Browse Source

feat(ecosystem): Include error level in Segment data (#44882)

So that we can filter out less worrying log levels in our segment
destinations, we include log level in the integration with Segment.
Chris Parsons 2 years ago
parent
commit
8580e1eebb

+ 2 - 0
src/sentry_plugins/segment/plugin.py

@@ -49,6 +49,7 @@ class SegmentPlugin(CorePluginMixin, DataForwardingPlugin):
             "eventId": event.event_id,
             "transaction": event.get_tag("transaction") or "",
             "release": event.get_tag("sentry:release") or "",
+            "level": event.get_tag("level") or "",
             "environment": event.get_tag("environment") or "",
         }
         if "sentry.interfaces.Http" in event.interfaces:
@@ -77,6 +78,7 @@ class SegmentPlugin(CorePluginMixin, DataForwardingPlugin):
             "eventId": event.event_id,
             "transaction": event.get_tag("transaction") or "",
             "release": event.get_tag("sentry:release") or "",
+            "level": event.get_tag("level") or "",
             "environment": event.get_tag("environment") or "",
         }
 

+ 1 - 0
tests/sentry_plugins/segment/test_plugin.py

@@ -48,6 +48,7 @@ class SegmentPluginTest(PluginTestCase):
                 "environment": "",
                 "eventId": event.event_id,
                 "exceptionType": "ValueError",
+                "level": "warning",
                 "release": "",
                 "transaction": "",
             },