fix(logging): An error should not have its own breadcrumb (getsentry#182)

untitaker · web-flow · commit 39b943877af9 · 2018-11-30T15:21:01.000+01:00
diff --git a/sentry_sdk/integrations/logging.py b/sentry_sdk/integrations/logging.py
@@ -54,15 +54,17 @@ def setup_once():
         old_callhandlers = logging.Logger.callHandlers
 
         def sentry_patched_callhandlers(self, record):
-            # This check is done twice, once also here before we even get
-            # the integration.  Otherwise we have a high chance of getting
-            # into a recursion error when the integration is resolved
-            # (this also is slower).
-            if record.name not in _IGNORED_LOGGERS:
-                integration = Hub.current.get_integration(LoggingIntegration)
-                if integration is not None:
-                    integration._handle_record(record)
-            return old_callhandlers(self, record)
+            try:
+                return old_callhandlers(self, record)
+            finally:
+                # This check is done twice, once also here before we even get
+                # the integration.  Otherwise we have a high chance of getting
+                # into a recursion error when the integration is resolved
+                # (this also is slower).
+                if record.name not in _IGNORED_LOGGERS:
+                    integration = Hub.current.get_integration(LoggingIntegration)
+                    if integration is not None:
+                        integration._handle_record(record)
 
         logging.Logger.callHandlers = sentry_patched_callhandlers
 
