chore: move exc deduplicator into client

let's keep event processors free for user and integration. It also feels
wrong to let scope.clear() disable such core functionality.

Author: untitaker

sentry_sdk/client.py

@@ -1,14 +1,16 @@
 import os
 import uuid
 
-from .utils import Dsn, SkipEvent
+from .utils import Dsn, SkipEvent, ContextVar
 from .transport import Transport
 from .consts import DEFAULT_OPTIONS, SDK_INFO
 from .stripping import strip_event, flatten_metadata
 
 
 NO_DSN = object()
 
+_most_recent_exception = ContextVar("sentry_most_recent_exception")
+
 
 class Client(object):
     def __init__(self, dsn=None, *args, **kwargs):
@@ -58,6 +60,11 @@ def _prepare_event(self, event, scope):
         if event.get("event_id") is None:
             event["event_id"] = uuid.uuid4().hex
 
+        if event._exc_value is not None:
+            if _most_recent_exception.get(None) is event._exc_value:
+                raise SkipEvent()
+            _most_recent_exception.set(event._exc_value)
+
         if scope is not None:
             scope.apply_to_event(event)
 

sentry_sdk/hub.py

@@ -4,7 +4,7 @@
 
 from ._compat import with_metaclass
 from .scope import Scope
-from .utils import Event, skip_internal_frames, ContextVar, DefaultEventProcessor
+from .utils import Event, skip_internal_frames, ContextVar
 
 
 _local = ContextVar("sentry_current_hub")
@@ -191,4 +191,3 @@ def _flush_event_processors(self):
 
 
 GLOBAL_HUB = Hub()
-GLOBAL_HUB.add_event_processor(DefaultEventProcessor)

sentry_sdk/utils.py

@@ -355,18 +355,6 @@ def __len__(self):
         return len(self._data)
 
 
-class DefaultEventProcessor(object):
-    def __init__(self):
-        self._most_recent_exception = ContextVar("most-recent-exception")
-
-    def __call__(self, event):
-        if event._exc_value is None:
-            return
-        if self._most_recent_exception.get(None) is event._exc_value:
-            raise SkipEvent()
-        self._most_recent_exception.set(event._exc_value)
-
-
 class SkipEvent(Exception):
     """Risen from an event processor to indicate that the event should be
     ignored and not be reported."""