Set HTTP client breadcrumb level based on status code

Author: sentrivana

sentry_sdk/integrations/aiohttp.py

@@ -27,6 +27,7 @@
     capture_internal_exceptions,
     ensure_integration_enabled,
     event_from_exception,
+    http_client_status_to_breadcrumb_level,
     logger,
     parse_url,
     parse_version,
@@ -277,13 +278,15 @@ async def on_request_end(session, trace_config_ctx, params):
             return
 
         span_data = trace_config_ctx.span_data or {}
-        span_data[SPANDATA.HTTP_STATUS_CODE] = int(params.response.status)
+        status_code = int(params.response.status)
+        span_data[SPANDATA.HTTP_STATUS_CODE] = status_code
         span_data["reason"] = params.response.reason
 
         sentry_sdk.add_breadcrumb(
             type="http",
             category="httplib",
             data=span_data,
+            level=http_client_status_to_breadcrumb_level(status_code),
         )
 
         span = trace_config_ctx.span

sentry_sdk/integrations/httpx.py

@@ -7,6 +7,7 @@
     SENSITIVE_DATA_SUBSTITUTE,
     capture_internal_exceptions,
     ensure_integration_enabled,
+    http_client_status_to_breadcrumb_level,
     logger,
     parse_url,
 )
@@ -101,6 +102,7 @@ def send(self, request, **kwargs):
                 type="http",
                 category="httplib",
                 data=data,
+                level=http_client_status_to_breadcrumb_level(rv.status_code),
             )
 
             return rv

sentry_sdk/integrations/stdlib.py

@@ -14,6 +14,7 @@
     capture_internal_exceptions,
     ensure_integration_enabled,
     get_current_thread_meta,
+    http_client_status_to_breadcrumb_level,
     is_sentry_url,
     logger,
     safe_repr,
@@ -144,14 +145,16 @@ def getresponse(self, *args, **kwargs):
             span_data[SPANDATA.HTTP_STATUS_CODE] = int(rv.status)
             span_data["reason"] = rv.reason
 
+            status_code = int(rv.status)
+            span.set_http_status(status_code)
+            span.set_data("reason", rv.reason)
+
             sentry_sdk.add_breadcrumb(
                 type="http",
                 category="httplib",
                 data=span_data,
+                level=http_client_status_to_breadcrumb_level(status_code),
             )
-
-            span.set_http_status(int(rv.status))
-            span.set_data("reason", rv.reason)
         finally:
             span.__exit__(None, None, None)
 

sentry_sdk/utils.py

@@ -1887,3 +1887,14 @@ def should_be_treated_as_error(ty, value):
         return False
 
     return True
+
+
+def http_client_status_to_breadcrumb_level(status_code):
+    # type: (Optional[int]) -> str
+    if status_code is not None:
+        if 500 <= status_code <= 599:
+            return "error"
+        elif 400 <= status_code <= 499:
+            return "warning"
+
+    return "info"