clean up sentry notification and avoid logger.exception when we do not want sentry
This commit is contained in:
parent
cf57be031f
commit
b9783ae0ed
|
@ -171,18 +171,30 @@ def set_user_sentry_context() -> None:
|
||||||
set_tag("username", username)
|
set_tag("username", username)
|
||||||
|
|
||||||
|
|
||||||
|
def should_notify_sentry(exception: Exception) -> bool:
|
||||||
|
"""Determine if we should notify sentry.
|
||||||
|
|
||||||
|
We want to capture_exception to log the exception to sentry, but we don't want to log:
|
||||||
|
1. ApiErrors that are just invalid tokens
|
||||||
|
2. NotAuthorizedError. we usually call check-permissions before calling an API to
|
||||||
|
make sure we'll have access, but there are some cases
|
||||||
|
where it's more convenient to just make the call from the frontend and handle the 403 appropriately.
|
||||||
|
"""
|
||||||
|
if isinstance(exception, ApiError):
|
||||||
|
if exception.error_code == "invalid_token":
|
||||||
|
return False
|
||||||
|
if isinstance(exception, NotAuthorizedError):
|
||||||
|
return False
|
||||||
|
return True
|
||||||
|
|
||||||
|
|
||||||
@api_error_blueprint.app_errorhandler(Exception) # type: ignore
|
@api_error_blueprint.app_errorhandler(Exception) # type: ignore
|
||||||
def handle_exception(exception: Exception) -> flask.wrappers.Response:
|
def handle_exception(exception: Exception) -> flask.wrappers.Response:
|
||||||
"""Handles unexpected exceptions."""
|
"""Handles unexpected exceptions."""
|
||||||
set_user_sentry_context()
|
set_user_sentry_context()
|
||||||
|
|
||||||
sentry_link = None
|
sentry_link = None
|
||||||
# we want to capture_exception to log the exception to sentry, but we don't want to log:
|
if should_notify_sentry(exception):
|
||||||
# 1. ApiErrors that are just invalid tokens
|
|
||||||
# 2. NotAuthorizedError
|
|
||||||
if (
|
|
||||||
not isinstance(exception, ApiError) or exception.error_code != "invalid_token"
|
|
||||||
) and not isinstance(exception, NotAuthorizedError):
|
|
||||||
id = capture_exception(exception)
|
id = capture_exception(exception)
|
||||||
|
|
||||||
if isinstance(exception, ApiError):
|
if isinstance(exception, ApiError):
|
||||||
|
@ -202,6 +214,12 @@ def handle_exception(exception: Exception) -> flask.wrappers.Response:
|
||||||
# seems to break the sentry sdk context where we no longer get back
|
# seems to break the sentry sdk context where we no longer get back
|
||||||
# an event id or send out tags like username
|
# an event id or send out tags like username
|
||||||
current_app.logger.exception(exception)
|
current_app.logger.exception(exception)
|
||||||
|
else:
|
||||||
|
current_app.logger.error(
|
||||||
|
f"Received exception: {exception}. Since we do not want this particular"
|
||||||
|
" exception in sentry, we cannot use logger.exception, so there will be no"
|
||||||
|
" backtrace. see api_error.py"
|
||||||
|
)
|
||||||
|
|
||||||
error_code = "internal_server_error"
|
error_code = "internal_server_error"
|
||||||
status_code = 500
|
status_code = 500
|
||||||
|
|
Loading…
Reference in New Issue