Browse Source

fix(utils.sdk): Disable SentryInternalFilter (#19564)

This filter invoked logic that is already being applied in the sentry sdk transport. As such it is not necessary to execute again. We also want to remove this filter because it impacts our ability to send to the upstream dsn (s4s), to which we can practically send anything.

Let's try this out first, and if it works let's remove the logging filter for real.
Markus Unterwaditzer 4 years ago
parent
commit
914e6e6ae2
1 changed files with 1 additions and 1 deletions
  1. 1 1
      src/sentry/utils/sdk.py

+ 1 - 1
src/sentry/utils/sdk.py

@@ -119,7 +119,7 @@ class SentryInternalFilter(logging.Filter):
     def filter(self, record):
     def filter(self, record):
         # TODO(mattrobenolt): handle an upstream Sentry
         # TODO(mattrobenolt): handle an upstream Sentry
         metrics.incr("internal.uncaptured.logs", skip_internal=False)
         metrics.incr("internal.uncaptured.logs", skip_internal=False)
-        return is_current_event_safe()
+        return True
 
 
 
 
 def configure_sdk():
 def configure_sdk():