feat(core): Add server.address to browser http.client spans

getsentry · Apr 16, 2024 · 5642fad · 5642fad
1 parent f1c4611
commit 5642fad
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/packages/browser/src/tracing/request.ts b/packages/browser/src/tracing/request.ts
@@ -4,6 +4,7 @@ import {
   addXhrInstrumentationHandler,
 } from '@sentry-internal/browser-utils';
 import {
+  SEMANTIC_ATTRIBUTE_SENTRY_OP,
   SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN,
   SentryNonRecordingSpan,
   getActiveSpan,
@@ -26,6 +27,7 @@ import {
   browserPerformanceTimeOrigin,
   dynamicSamplingContextToSentryBaggageHeader,
   generateSentryTraceHeader,
+  parseUrl,
   stringMatchesSomePattern,
 } from '@sentry/utils';
 import { WINDOW } from '../helpers';
@@ -310,6 +312,8 @@ export function xhrCallback(
 
   const hasParent = !!getActiveSpan();
 
+  const parsedUrl = parseUrl(sentryXhrData.url);
+
   const span =
     shouldCreateSpanResult && hasParent
       ? startInactiveSpan({
@@ -318,9 +322,10 @@ export function xhrCallback(
             type: 'xhr',
             'http.method': sentryXhrData.method,
             url: sentryXhrData.url,
+            'server.address': parsedUrl.host,
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.browser',
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'http.client',
           },
-          op: 'http.client',
         })
       : new SentryNonRecordingSpan();
 

diff --git a/packages/core/src/fetch.ts b/packages/core/src/fetch.ts
@@ -4,9 +4,10 @@ import {
   dynamicSamplingContextToSentryBaggageHeader,
   generateSentryTraceHeader,
   isInstanceOf,
+  parseUrl,
 } from '@sentry/utils';
 import { getClient, getCurrentScope, getIsolationScope } from './currentScopes';
-import { SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN } from './semanticAttributes';
+import { SEMANTIC_ATTRIBUTE_SENTRY_OP, SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN } from './semanticAttributes';
 import {
   SPAN_STATUS_ERROR,
   getDynamicSamplingContextFromClient,
@@ -83,6 +84,8 @@ export function instrumentFetchRequest(
 
   const hasParent = !!getActiveSpan();
 
+  const parsedUrl = parseUrl(url);
+
   const span =
     shouldCreateSpanResult && hasParent
       ? startInactiveSpan({
@@ -91,9 +94,10 @@ export function instrumentFetchRequest(
             url,
             type: 'fetch',
             'http.method': method,
+            'server.address': parsedUrl.host,
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: spanOrigin,
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'http.client',
           },
-          op: 'http.client',
         })
       : new SentryNonRecordingSpan();