log debug

getsentry · Nov 15, 2024 · ea95b59 · ea95b59
1 parent be740a7
commit ea95b59
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 7 deletions.
diff --git a/relay-server/src/envelope.rs b/relay-server/src/envelope.rs
@@ -119,7 +119,7 @@ pub enum ItemType {
     CheckIn,
     /// A standalone span.
     Span,
-    /// A standalone OpenTelemetry span.
+    /// A standalone OpenTelemetry span serialized as JSON.
     OtelSpan,
     /// An OTLP TracesData container.
     OtelTracesData,

diff --git a/relay-server/src/services/processor/span.rs b/relay-server/src/services/processor/span.rs
@@ -79,12 +79,20 @@ fn track_invalid(managed_envelope: &mut TypedEnvelope<SpanGroup>, reason: Discar
 
 fn parse_traces_data(item: Item) -> Result<TracesData, DiscardReason> {
     match item.content_type() {
-        Some(&ContentType::Json) => {
-            serde_json::from_slice(&item.payload()).map_err(|_| DiscardReason::InvalidJson)
-        }
-        Some(&ContentType::Protobuf) => {
-            TracesData::decode(item.payload()).map_err(|_| DiscardReason::InvalidProtobuf)
-        }
+        Some(&ContentType::Json) => serde_json::from_slice(&item.payload()).map_err(|e| {
+            relay_log::debug!(
+                error = &e as &dyn std::error::Error,
+                "Failed to parse traces data as JSON"
+            );
+            DiscardReason::InvalidJson
+        }),
+        Some(&ContentType::Protobuf) => TracesData::decode(item.payload()).map_err(|e| {
+            relay_log::debug!(
+                error = &e as &dyn std::error::Error,
+                "Failed to parse traces data as protobuf"
+            );
+            DiscardReason::InvalidProtobuf
+        }),
         _ => Err(DiscardReason::ContentType),
     }
 }