metrics(processor): emit counter for feedback screenshots

relay-server/src/services/processor.rs

@@ -1496,6 +1496,7 @@ impl EnvelopeProcessorService {
         if state.has_event() {
             event::scrub(state)?;
             event::serialize(state)?;
+            event::emit_feedback_metrics(state.envelope());
         }
 
         attachment::scrub(state);

relay-server/src/services/processor/event.rs

@@ -20,7 +20,7 @@ use relay_quotas::DataCategory;
 use relay_statsd::metric;
 use serde_json::Value as SerdeValue;
 
-use crate::envelope::{AttachmentType, ContentType, Item, ItemType};
+use crate::envelope::{AttachmentType, ContentType, Envelope, Item, ItemType};
 use crate::extractors::RequestMeta;
 use crate::services::outcome::Outcome;
 use crate::services::processor::{
@@ -420,6 +420,26 @@ pub fn has_unprintable_fields(event: &Annotated<Event>) -> bool {
     }
 }
 
+//TODO: comment
+//TODO: add test: all incr/no incr cases, envelope unchanged
+pub fn emit_feedback_metrics(envelope: &Envelope) -> i64 {
+    let mut has_feedback = false;
+    let mut num_attachments = 0;
+    for item in envelope.items() {
+        match item.ty() {
+            ItemType::UserReportV2 => has_feedback = true,
+            ItemType::Attachment => num_attachments += 1,
+            _ => (),
+        }
+    }
+    if has_feedback && num_attachments > 0 {
+        metric!(counter(RelayCounters::FeedbackAttachments) += num_attachments);
+        num_attachments
+    } else {
+        0
+    }
+}
+
 /// Checks for duplicate items in an envelope.
 ///
 /// An item is considered duplicate if it was not removed by sanitation in `process_event` and

relay-server/src/statsd.rs

@@ -678,6 +678,9 @@ pub enum RelayCounters {
     /// Counter for when the DSC is missing from an event that comes from an SDK that should support
     /// it.
     MissingDynamicSamplingContext,
+    /// The number of attachments processed in the same envelope as a user_report_v2 event.
+    #[cfg(feature = "processing")]
+    FeedbackAttachments,
 }
 
 impl CounterMetric for RelayCounters {
@@ -720,6 +723,8 @@ impl CounterMetric for RelayCounters {
             #[cfg(feature = "processing")]
             RelayCounters::TransactionsFromSpans => "transactions_from_spans",
             RelayCounters::MissingDynamicSamplingContext => "missing_dynamic_sampling_context",
+            #[cfg(feature = "processing")]
+            RelayCounters::FeedbackAttachments => "processing.feedback_attachments",
         }
     }
 }