fix(core): Don't return trace data in getTraceData and getTraceMetaTags if SDK is disabled

dev-packages/node-integration-tests/suites/tracing/meta-tags/test.ts

@@ -5,7 +5,7 @@ describe('getTraceMetaTags', () => {
     cleanupChildProcesses();
   });
 
-  test('injects sentry tracing <meta> tags', async () => {
+  test('injects <meta> tags with trace from incoming headers', async () => {
     const traceId = 'cd7ee7a6fe3ebe7ab9c3271559bc203c';
     const parentSpanId = '100ff0980e7a4ead';
 
@@ -22,4 +22,53 @@ describe('getTraceMetaTags', () => {
     expect(html).toMatch(/<meta name="sentry-trace" content="cd7ee7a6fe3ebe7ab9c3271559bc203c-[a-z0-9]{16}-1"\/>/);
     expect(html).toContain('<meta name="baggage" content="sentry-environment=production"/>');
   });
+
+  test('injects <meta> tags with new trace if no incoming headers', async () => {
+    const runner = createRunner(__dirname, 'server.js').start();
+
+    const response = await runner.makeRequest('get', '/test');
+
+    // @ts-ignore - response is defined, types just don't reflect it
+    const html = response?.response as unknown as string;
+
+    const traceId = html.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-1"\/>/)?.[1];
+    expect(traceId).not.toBeUndefined();
+
+    expect(html).toContain('<meta name="baggage"');
+    expect(html).toContain(`sentry-trace_id=${traceId}`);
+  });
+
+  test('injects <meta> tags with negative sampling decision if tracesSampleRate is 0', async () => {
+    const runner = createRunner(__dirname, 'server-tracesSampleRate-zero.js').start();
+
+    const response = await runner.makeRequest('get', '/test');
+
+    // @ts-ignore - response is defined, types just don't reflect it
+    const html = response?.response as unknown as string;
+
+    const traceId = html.match(/<meta name="sentry-trace" content="([a-z0-9]{32})-[a-z0-9]{16}-0"\/>/)?.[1];
+    expect(traceId).not.toBeUndefined();
+
+    expect(html).toContain('<meta name="baggage"');
+    expect(html).toContain(`sentry-trace_id=${traceId}`);
+    expect(html).toContain('sentry-sampled=false');
+  });
+
+  test("doesn't inject sentry tracing <meta> tags if SDK is disabled", async () => {
+    const traceId = 'cd7ee7a6fe3ebe7ab9c3271559bc203c';
+    const parentSpanId = '100ff0980e7a4ead';
+
+    const runner = createRunner(__dirname, 'server-sdk-disabled.js').start();
+
+    const response = await runner.makeRequest('get', '/test', {
+      'sentry-trace': `${traceId}-${parentSpanId}-1`,
+      baggage: 'sentry-environment=production',
+    });
+
+    // @ts-ignore - response is defined, types just don't reflect it
+    const html = response?.response as unknown as string;
+
+    expect(html).not.toContain('"sentry-trace"');
+    expect(html).not.toContain('"baggage"');
+  });
 });

packages/core/src/utils/traceData.ts

@@ -8,6 +8,7 @@ import {
 import { getAsyncContextStrategy } from '../asyncContext';
 import { getMainCarrier } from '../carrier';
 import { getClient, getCurrentScope } from '../currentScopes';
+import { isEnabled } from '../exports';
 import { getDynamicSamplingContextFromClient, getDynamicSamplingContextFromSpan } from '../tracing';
 import { getActiveSpan, getRootSpan, spanToTraceHeader } from './spanUtils';
 
@@ -23,6 +24,10 @@ import { getActiveSpan, getRootSpan, spanToTraceHeader } from './spanUtils';
  * or meta tag name.
  */
 export function getTraceData(): SerializedTraceData {
+  if (!isEnabled()) {
+    return {};
+  }
+
   const carrier = getMainCarrier();
   const acs = getAsyncContextStrategy(carrier);
   if (acs.getTraceData) {

packages/core/test/lib/utils/traceData.test.ts

@@ -1,5 +1,6 @@
 import { SentrySpan, getTraceData } from '../../../src/';
 import * as SentryCoreCurrentScopes from '../../../src/currentScopes';
+import * as SentryCoreExports from '../../../src/exports';
 import * as SentryCoreTracing from '../../../src/tracing';
 import * as SentryCoreSpanUtils from '../../../src/utils/spanUtils';
 
@@ -22,6 +23,14 @@ const mockedScope = {
 } as any;
 
 describe('getTraceData', () => {
+  beforeEach(() => {
+    jest.spyOn(SentryCoreExports, 'isEnabled').mockReturnValue(true);
+  });
+
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
   it('returns the tracing data from the span, if a span is available', () => {
     {
       jest.spyOn(SentryCoreTracing, 'getDynamicSamplingContextFromSpan').mockReturnValueOnce({
@@ -139,6 +148,14 @@ describe('getTraceData', () => {
 
     expect(traceData).toEqual({});
   });
+
+  it('returns an empty object if the SDK is disabled', () => {
+    jest.spyOn(SentryCoreExports, 'isEnabled').mockReturnValueOnce(false);
+
+    const traceData = getTraceData();
+
+    expect(traceData).toEqual({});
+  });
 });
 
 describe('isValidBaggageString', () => {