Slim down outgoing payload

Author: sentrivana

sentry_sdk/opentelemetry/span_processor.py

@@ -45,6 +45,11 @@
 
 DEFAULT_MAX_SPANS = 1000
 
+# Span attributes internal to the SDK that should be omitted from the final payload
+IGNORE_SPAN_ATTRIBUTES = [
+    "flag.count",
+]
+
 
 class SentrySpanProcessor(SpanProcessor):
     """
@@ -289,8 +294,11 @@ def _span_to_json(self, span):
         if parent_span_id:
             span_json["parent_span_id"] = parent_span_id
 
-        if span.attributes:
-            span_json["data"] = dict(span.attributes)
+        if getattr(span, "attributes", {}):
+            span_json["data"] = {}
+            for key, value in span.attributes.items():
+                if key not in IGNORE_SPAN_ATTRIBUTES:
+                    span_json["data"][key] = value
 
         return span_json
 

tests/test_feature_flags.py

@@ -259,3 +259,19 @@ def test_flag_limit(sentry_init, capture_events):
         }
     )
     assert "flag.evaluation.10" not in event["spans"][0]["data"]
+
+
+def test_flag_counter_not_sent(sentry_init, capture_events):
+    sentry_init(traces_sample_rate=1.0)
+
+    events = capture_events()
+
+    with start_transaction(name="hi"):
+        with start_span(op="foo", name="bar"):
+            add_feature_flag("0", True)
+            add_feature_flag("1", True)
+            add_feature_flag("2", True)
+            add_feature_flag("3", True)
+
+    (event,) = events
+    assert "flag.counter" not in event["spans"][0]["data"]