feat(sentry): setup before_send to ignore global id errors by message

KK-1395.

The ignore_messages option in sentry_sdk would be great, but since the
Graphene raises a general Exception from invalid global ids, we need to
follow the exception message instead of class.

kukkuu/settings.py

@@ -149,16 +149,23 @@
 def sentry_before_send(event, hint):
     """
     Ignore certain errors from being sent to Sentry.
+    Some Graphene errors are generic Exceptions that are not useful to log.
+    Since they are generic, we need to ignore them by their message.
     NOTE: The `ignore_errors` option is actually deprecated,
     nevertheless it's included in the init configuration.
     See https://github.com/getsentry/sentry-python/issues/149
     """
 
     IGNORED_ERRORS_CLASSES = (ExpiredSignatureError, AuthenticationExpiredError)
+    IGNORED_ERROR_MESSAGES = [
+        "Unable to parse global ID",
+    ]
     if "exc_info" in hint:
         exc_type, exc_value, traceback = hint["exc_info"]
         if isinstance(exc_value, IGNORED_ERRORS_CLASSES):
             return None
+        if any(msg in str(exc_value) for msg in IGNORED_ERROR_MESSAGES):
+            return None
     return event
 
 

kukkuu/tests/test_sentry.py

@@ -5,6 +5,8 @@
 from kukkuu.settings import sentry_before_send
 
 test_cases = [
+    (Exception("Unable to parse global ID."), True),
+    (Exception('Unable to parse global ID "some_id".'), True),
     (ExpiredSignatureError("Expired signature"), True),
     (AuthenticationExpiredError("Authentication expired"), True),
     (Exception("Some other error"), False),