From 6cc36030d3a1cc3455dc43c0b7bc665d096d3247 Mon Sep 17 00:00:00 2001 From: Hiroshi Nishio Date: Tue, 24 Feb 2026 15:11:43 -0800 Subject: [PATCH] Downgrade logger.error to logger.warning for server errors in _handle_generic_error Sentry's LoggingIntegration auto-captures ERROR-level logs as events, so logger.error() was sending server errors to Sentry even though capture_exception() was explicitly skipped for 5xx errors. --- utils/error/handle_exceptions.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/utils/error/handle_exceptions.py b/utils/error/handle_exceptions.py index 34897b422..da84e1d9b 100644 --- a/utils/error/handle_exceptions.py +++ b/utils/error/handle_exceptions.py @@ -164,9 +164,10 @@ def _handle_generic_error( err_msg = f"{func_name} encountered an {type(err).__name__}: {err}\n\nArgs: {json.dumps(log_args, indent=2, default=str)}\n\nKwargs: {json.dumps(log_kwargs, indent=2, default=str)}" if is_server_error(err): logger.warning("%s received server error, not reporting to Sentry", func_name) + logger.warning(err_msg) else: sentry_sdk.capture_exception(err) - logger.error(err_msg) + logger.error(err_msg) if raise_on_error: raise err return error_return