Set http breadcrumb level based on response code (#940)

cleptric · web-flow · commit 14e828dc1200 · 2024-09-04T09:40:22.000+02:00
diff --git a/src/Sentry/Laravel/Features/HttpClientIntegration.php b/src/Sentry/Laravel/Features/HttpClientIntegration.php
@@ -117,7 +117,9 @@ public function handleResponseReceivedHandlerForBreadcrumb(ResponseReceived $eve
     {
         $level = Breadcrumb::LEVEL_INFO;
 
-        if ($event->response->failed()) {
+        if ($event->response->clientError()) {
+            $level = Breadcrumb::LEVEL_WARNING;
+        } elseif ($event->response->serverError()) {
             $level = Breadcrumb::LEVEL_ERROR;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -117,7 +117,9 @@ public function handleResponseReceivedHandlerForBreadcrumb(ResponseReceived $eve`
`117`	`117`	`{`
`118`	`118`	`$level = Breadcrumb::LEVEL_INFO;`
`119`	`119`
`120`		`- if ($event->response->failed()) {`
	`120`	`+ if ($event->response->clientError()) {`
	`121`	`+ $level = Breadcrumb::LEVEL_WARNING;`
	`122`	`+ } elseif ($event->response->serverError()) {`
`121`	`123`	`$level = Breadcrumb::LEVEL_ERROR;`
`122`	`124`	`}`
`123`	`125`