use isolationScope for consistency, guard for defaultIsolationScope

getsentry · Apr 9, 2024 · d7e536e · d7e536e
1 parent 1d001ed
commit d7e536e
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/packages/node/src/integrations/tracing/hapi/index.ts b/packages/node/src/integrations/tracing/hapi/index.ts
@@ -7,9 +7,13 @@ import {
   defineIntegration,
   getActiveSpan,
   getCurrentScope,
+  getDefaultIsolationScope,
+  getIsolationScope,
   getRootSpan,
 } from '@sentry/core';
 import type { IntegrationFn } from '@sentry/types';
+import { logger } from '@sentry/utils';
+import { DEBUG_BUILD } from '../../../debug-build';
 import type { Boom, RequestEvent, ResponseObject, Server } from './types';
 
 const _hapiIntegration = (() => {
@@ -59,9 +63,14 @@ export const hapiErrorPlugin = {
     const server = serverArg as unknown as Server;
 
     server.events.on('request', (request, event) => {
-      const route = request.route;
-      if (route && route.path) {
-        getCurrentScope().setTransactionName(`${route.method?.toUpperCase() || 'GET'} ${route.path}`);
+      if (getIsolationScope() !== getDefaultIsolationScope()) {
+        const route = request.route;
+        if (route && route.path) {
+          getIsolationScope().setTransactionName(`${route.method?.toUpperCase() || 'GET'} ${route.path}`);
+        }
+      } else {
+        DEBUG_BUILD &&
+          logger.warn('Isolation scope is still the default isolation scope - skipping setting transactionName');
       }
 
       const activeSpan = getActiveSpan();