set url source

Author: chargome

packages/core/src/tracing/spans/captureSpan.ts

@@ -248,6 +248,11 @@ function inferHttpSpanData(
   if (typeof httpRoute === 'string') {
     spanJSON.name = `${httpMethod} ${httpRoute}`;
     safeSetSpanJSONAttributes(spanJSON, { [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'route' });
+  } else {
+    // Fallback: set source to 'url' for HTTP spans without a route.
+    // The spec requires sentry.span.source on segment spans, and the non-streamed exporter
+    // always sets this — so we need to ensure it's present for streamed spans too.
+    safeSetSpanJSONAttributes(spanJSON, { [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'url' });
   }
 }
 

packages/core/test/lib/tracing/spans/captureSpan.test.ts

@@ -530,10 +530,11 @@ describe('inferSpanDataFromOtelAttributes', () => {
       expect(spanJSON.attributes?.['sentry.source']).toBe('route');
     });
 
-    it('does not overwrite name when no http.route', () => {
+    it('does not overwrite name when no http.route but sets source to url', () => {
       const spanJSON = makeSpanJSON('GET', { 'http.request.method': 'GET', 'url.full': 'http://example.com/api' });
       inferSpanDataFromOtelAttributes(spanJSON, 2);
       expect(spanJSON.name).toBe('GET');
+      expect(spanJSON.attributes?.['sentry.source']).toBe('url');
     });
 
     it('does not overwrite sentry.op if already set', () => {