fix(node): Ensure tracing is suppressed for all generated requests

This exports a new `suppressTracing` utility from `@sentry/opentelemetry` which can be used to wrap a callback to avoid tracing inside of it. We use this for bun transport, and for nextjs error symbolication.

Author: mydea

packages/bun/package.json

@@ -44,6 +44,7 @@
   "dependencies": {
     "@sentry/core": "8.0.0-alpha.9",
     "@sentry/node": "8.0.0-alpha.9",
+    "@sentry/opentelemetry": "8.0.0-alpha.9",
     "@sentry/types": "8.0.0-alpha.9",
     "@sentry/utils": "8.0.0-alpha.9"
   },

packages/bun/src/transports/index.ts

@@ -1,4 +1,5 @@
 import { createTransport } from '@sentry/core';
+import { suppressTracing } from '@sentry/opentelemetry';
 import type { BaseTransportOptions, Transport, TransportMakeRequestResponse, TransportRequest } from '@sentry/types';
 import { rejectedSyncPromise } from '@sentry/utils';
 
@@ -19,14 +20,16 @@ export function makeFetchTransport(options: BunTransportOptions): Transport {
     };
 
     try {
-      return fetch(options.url, requestOptions).then(response => {
-        return {
-          statusCode: response.status,
-          headers: {
-            'x-sentry-rate-limits': response.headers.get('X-Sentry-Rate-Limits'),
-            'retry-after': response.headers.get('Retry-After'),
-          },
-        };
+      return suppressTracing(() => {
+        return fetch(options.url, requestOptions).then(response => {
+          return {
+            statusCode: response.status,
+            headers: {
+              'x-sentry-rate-limits': response.headers.get('X-Sentry-Rate-Limits'),
+              'retry-after': response.headers.get('Retry-After'),
+            },
+          };
+        });
       });
     } catch (e) {
       return rejectedSyncPromise(e);

packages/nextjs/src/common/devErrorSymbolicationEventProcessor.ts

@@ -1,3 +1,4 @@
+import { suppressTracing } from '@sentry/opentelemetry';
 import type { Event, EventHint } from '@sentry/types';
 import { GLOBAL_OBJ } from '@sentry/utils';
 import type { StackFrame } from 'stacktrace-parser';
@@ -40,17 +41,19 @@ async function resolveStackFrame(
 
     const controller = new AbortController();
     const timer = setTimeout(() => controller.abort(), 3000);
-    const res = await fetch(
-      `${
-        // eslint-disable-next-line no-restricted-globals
-        typeof window === 'undefined' ? 'http://localhost:3000' : '' // TODO: handle the case where users define a different port
-      }${basePath}/__nextjs_original-stack-frame?${params.toString()}`,
-      {
-        signal: controller.signal,
-      },
-    ).finally(() => {
-      clearTimeout(timer);
-    });
+    const res = await suppressTracing(() =>
+      fetch(
+        `${
+          // eslint-disable-next-line no-restricted-globals
+          typeof window === 'undefined' ? 'http://localhost:3000' : '' // TODO: handle the case where users define a different port
+        }${basePath}/__nextjs_original-stack-frame?${params.toString()}`,
+        {
+          signal: controller.signal,
+        },
+      ).finally(() => {
+        clearTimeout(timer);
+      }),
+    );
 
     if (!res.ok || res.status === 204) {
       return null;

packages/node/src/transports/http.ts

@@ -2,9 +2,8 @@ import * as http from 'node:http';
 import * as https from 'node:https';
 import { Readable } from 'stream';
 import { createGzip } from 'zlib';
-import { context } from '@opentelemetry/api';
-import { suppressTracing } from '@opentelemetry/core';
 import { createTransport } from '@sentry/core';
+import { suppressTracing } from '@sentry/opentelemetry';
 import type {
   BaseTransportOptions,
   Transport,
@@ -82,7 +81,7 @@ export function makeNodeTransport(options: NodeTransportOptions): Transport {
     : new nativeHttpModule.Agent({ keepAlive, maxSockets: 30, timeout: 2000 });
 
   // This ensures we do not generate any spans in OpenTelemetry for the transport
-  return context.with(suppressTracing(context.active()), () => {
+  return suppressTracing(() => {
     const requestExecutor = createRequestExecutor(options, options.httpModule ?? nativeHttpModule, agent);
     return createTransport(options, requestExecutor);
   });

packages/opentelemetry/src/index.ts

@@ -23,6 +23,8 @@ export { isSentryRequestSpan } from './utils/isSentryRequest';
 export { getActiveSpan } from './utils/getActiveSpan';
 export { startSpan, startSpanManual, startInactiveSpan, withActiveSpan, continueTrace } from './trace';
 
+export { suppressTracing } from './utils/suppressTracing';
+
 // eslint-disable-next-line deprecation/deprecation
 export { setupGlobalHub } from './custom/hub';
 // eslint-disable-next-line deprecation/deprecation

packages/opentelemetry/src/utils/suppressTracing.ts

@@ -0,0 +1,8 @@
+import { context } from '@opentelemetry/api';
+import { suppressTracing as suppressTracingImpl } from '@opentelemetry/core';
+
+/** Suppress tracing in the given callback, ensuring no spans are generated inside of it. */
+export function suppressTracing<T>(callback: () => T): T {
+  const ctx = suppressTracingImpl(context.active());
+  return context.with(ctx, callback);
+}