filtering logic

Author: Luca Forstner

dev-packages/e2e-tests/test-applications/node-exports-test-app/scripts/consistentExports.ts

@@ -68,6 +68,22 @@ const DEPENDENTS: Dependent[] = [
     // Next.js doesn't require explicit exports, so we can just merge top level and `default` exports:
     // @ts-expect-error: `default` is not in the type definition but it's defined
     exports: Object.keys({ ...SentryNextJs, ...SentryNextJs.default }),
+    ignoreExports: [
+      // legacy, to be removed...
+      'Handlers',
+      // legacy, to be removed...
+      'Integrations',
+      // legacy, to be removed...
+      'addGlobalEventProcessor',
+      // legacy, to be removed...
+      'autoDiscoverNodePerformanceMonitoringIntegrations',
+      // legacy, to be removed...
+      'getActiveTransaction',
+      // legacy, to be removed...
+      'hapiErrorPlugin',
+      // legacy, to be removed...
+      'makeMain',
+    ],
   },
   {
     package: '@sentry/remix',

packages/nextjs/src/server/index.ts

@@ -18,6 +18,7 @@ export { onUncaughtExceptionIntegration } from './onUncaughtExceptionIntegration
 
 const globalWithInjectedValues = GLOBAL_OBJ as typeof GLOBAL_OBJ & {
   __rewriteFramesDistDir__?: string;
+  __sentryRewritesTunnelPath__?: string;
 };
 
 /**
@@ -73,7 +74,9 @@ export function init(options: NodeOptions): void {
   }
 
   const customDefaultIntegrations = [
-    ...getDefaultIntegrations(options).filter(integration => integration.name !== 'OnUncaughtException'),
+    ...getDefaultIntegrations(options).filter(
+      integration => integration.name !== 'OnUncaughtException' && integration.name !== 'NodeFetch',
+    ),
     onUncaughtExceptionIntegration(),
   ];
 
@@ -107,19 +110,49 @@ export function init(options: NodeOptions): void {
 
   nodeInit(opts);
 
-  const filterTransactions: EventProcessor = event => {
-    return event.type === 'transaction' && event.transaction === '/404' ? null : event;
+  const filterLowQualityTransactions: EventProcessor = event => {
+    if (event.type === 'transaction') {
+      if (event.transaction?.match(/GET \/.*\/static\/.*.js/)) {
+        // TODO: Make this based on basePath and assetPrefix
+        return null;
+      } else if (
+        globalWithInjectedValues.__sentryRewritesTunnelPath__ &&
+        event.transaction === `POST ${globalWithInjectedValues.__sentryRewritesTunnelPath__}`
+      ) {
+        return null;
+      } else if (event.transaction?.match(/\/__nextjs_original-stack-frame/)) {
+        return null;
+      } else if (event.transaction === '/404') {
+        return null;
+      }
+
+      return event;
+    } else {
+      return event;
+    }
   };
-
-  filterTransactions.id = 'NextServer404TransactionFilter';
+  filterLowQualityTransactions.id = 'NextLowQualityTransactionsFilter';
+  addEventProcessor(filterLowQualityTransactions);
+
+  const filterSentrySpans: EventProcessor = event => {
+    if (event.type === 'transaction') {
+      event.spans = event.spans?.filter(
+        span =>
+          !span.data?.['http.target']?.includes('sentry_client') && !span.data?.['http.target']?.includes('sentry_key'),
+      );
+      event.spans = event.spans?.filter(span => !span.data?.['http.url']?.includes('__nextjs_original-stack-frame'));
+    }
+
+    return event;
+  };
+  filterSentrySpans.id = 'NextFilterSentrySpans';
+  addEventProcessor(filterSentrySpans);
 
   setTag('runtime', 'node');
   if (IS_VERCEL) {
     setTag('vercel', true);
   }
 
-  addEventProcessor(filterTransactions);
-
   if (process.env.NODE_ENV === 'development') {
     addEventProcessor(devErrorSymbolicationEventProcessor);
   }