fix(core): Set log level for Fetch/XHR breadcrumbs based on status code

Signed-off-by: Kaung Zin Hein <[email protected]>

Author: Zen-cronic

packages/core/src/breadcrumbs.ts

@@ -1,6 +1,7 @@
 import type { Breadcrumb, BreadcrumbHint } from '@sentry/types';
 import { consoleSandbox, dateTimestampInSeconds } from '@sentry/utils';
 import { getClient, getIsolationScope } from './currentScopes';
+import { assignBreadcrumbLogLevel } from './utils/breadcrumbsUtils';
 
 /**
  * Default maximum number of breadcrumbs added to an event. Can be overwritten
@@ -25,7 +26,7 @@ export function addBreadcrumb(breadcrumb: Breadcrumb, hint?: BreadcrumbHint): vo
   if (maxBreadcrumbs <= 0) return;
 
   const timestamp = dateTimestampInSeconds();
-  const mergedBreadcrumb = { timestamp, ...breadcrumb };
+  const mergedBreadcrumb = { timestamp, ...assignBreadcrumbLogLevel(breadcrumb) };
   const finalBreadcrumb = beforeBreadcrumb
     ? (consoleSandbox(() => beforeBreadcrumb(mergedBreadcrumb, hint)) as Breadcrumb | null)
     : mergedBreadcrumb;

packages/core/src/scope.ts

@@ -24,6 +24,7 @@ import type {
 import { dateTimestampInSeconds, generatePropagationContext, isPlainObject, logger, uuid4 } from '@sentry/utils';
 
 import { updateSession } from './session';
+import { assignBreadcrumbLogLevel } from './utils/breadcrumbsUtils';
 import { _getSpanForScope, _setSpanForScope } from './utils/spanOnScope';
 
 /**
@@ -412,7 +413,7 @@ class ScopeClass implements ScopeInterface {
 
     const mergedBreadcrumb = {
       timestamp: dateTimestampInSeconds(),
-      ...breadcrumb,
+      ...assignBreadcrumbLogLevel(breadcrumb),
     };
 
     const breadcrumbs = this._breadcrumbs;

packages/core/src/utils/breadcrumbsUtils.ts

@@ -0,0 +1,20 @@
+import type { Breadcrumb } from '@sentry/types';
+
+/**
+ * Set a Fetch/XHR breadcrumb's log level based on the returned status code
+ * @param breadcrumb
+ */
+export function assignBreadcrumbLogLevel(breadcrumb: Breadcrumb): Breadcrumb {
+  const statusCode = breadcrumb?.data?.status_code;
+  if (typeof statusCode !== 'number') {
+    return breadcrumb;
+  }
+
+  if (statusCode >= 400 && statusCode < 500) {
+    breadcrumb.level = 'warning';
+  } else if (statusCode >= 500) {
+    breadcrumb.level = 'error';
+  }
+
+  return breadcrumb;
+}