fix: Only set last_event_id when event actually captured

untitaker · untitaker · commit 8e711dc970ee · 2018-09-29T20:33:39.000+02:00
diff --git a/sentry_sdk/client.py b/sentry_sdk/client.py
@@ -153,10 +153,12 @@ def capture_event(self, event, hint=None, scope=None):
         rv = event.get("event_id")
         if rv is None:
             event["event_id"] = rv = uuid.uuid4().hex
-        if self._should_capture(event, hint, scope):
-            event = self._prepare_event(event, hint, scope)
-            if event is not None:
-                self.transport.capture_event(event)
+        if not self._should_capture(event, hint, scope):
+            return
+        event = self._prepare_event(event, hint, scope)
+        if event is None:
+            return
+        self.transport.capture_event(event)
         return rv
 
     def close(self, timeout=None, shutdown_callback=None):
