fix(nestjs): Ensure Nest.js interceptor works with non-http context (#…

…11880) Fixes #11877
getsentry · May 3, 2024 · fd10a2b · fd10a2b
1 parent 56197af
commit fd10a2b
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/packages/node/src/integrations/tracing/nest.ts b/packages/node/src/integrations/tracing/nest.ts
@@ -5,6 +5,8 @@ import type { IntegrationFn } from '@sentry/types';
 import { logger } from '@sentry/utils';
 
 interface MinimalNestJsExecutionContext {
+  getType: () => string;
+
   switchToHttp: () => {
     // minimal request object
     // according to official types, all properties are required but
@@ -57,10 +59,13 @@ export function setupNestErrorHandler(app: MinimalNestJsApp, baseFilter: NestJsE
         return next.handle();
       }
 
-      const req = context.switchToHttp().getRequest();
-      if (req.route) {
-        getIsolationScope().setTransactionName(`${req.method?.toUpperCase() || 'GET'} ${req.route.path}`);
+      if (context.getType() === 'http') {
+        const req = context.switchToHttp().getRequest();
+        if (req.route) {
+          getIsolationScope().setTransactionName(`${req.method?.toUpperCase() || 'GET'} ${req.route.path}`);
+        }
       }
+
       return next.handle();
     },
   });