ref(node): Stop using registerSpanErrorInstrumentation() on server

Author: mydea

packages/core/src/server-runtime-client.ts

@@ -3,7 +3,6 @@ import { Client } from './client';
 import { getIsolationScope } from './currentScopes';
 import { DEBUG_BUILD } from './debug-build';
 import type { Scope } from './scope';
-import { registerSpanErrorInstrumentation } from './tracing';
 import { DEFAULT_TRANSPORT_BUFFER_SIZE } from './transports/base';
 import { addUserAgentToTransportHeaders } from './transports/userAgent';
 import type { CheckIn, MonitorConfig, SerializedCheckIn } from './types/checkin';
@@ -38,9 +37,6 @@ export class ServerRuntimeClient<
    * @param options Configuration options for this SDK.
    */
   public constructor(options: O) {
-    // Server clients always support tracing
-    registerSpanErrorInstrumentation();
-
     addUserAgentToTransportHeaders(options);
 
     super(options);

packages/core/src/tracing/errors.ts

@@ -33,10 +33,6 @@ export function registerSpanErrorInstrumentation(): void {
     }
   }
 
-  // The function name will be lost when bundling but we need to be able to identify this listener later to maintain the
-  // node.js default exit behaviour
-  errorCallback.tag = 'sentry_tracingErrorCallback';
-
   errorsInstrumented = true;
   addGlobalErrorInstrumentationHandler(errorCallback);
   addGlobalUnhandledRejectionInstrumentationHandler(errorCallback);

packages/node-core/src/integrations/onuncaughtexception.ts

@@ -89,8 +89,6 @@ export function makeErrorHandler(client: NodeClient, options: OnUncaughtExceptio
           return (
             // as soon as we're using domains this listener is attached by node itself
             listener.name !== 'domainUncaughtExceptionClear' &&
-            // the handler we register for tracing
-            listener.tag !== 'sentry_tracingErrorCallback' &&
             // the handler we register in this integration
             (listener as ErrorHandler)._errorHandler !== true
           );