tests

dev-packages/e2e-tests/test-applications/nextjs-15/app/pageload-tracing/page.tsx

@@ -6,7 +6,7 @@ export default async function Page() {
 }
 
 export async function generateMetadata() {
-  (await fetch('http://example.com/')).text();
+  (await fetch('http://example.com/', { cache: 'no-store' })).text();
 
   return {
     title: 'my title',

dev-packages/e2e-tests/test-applications/nextjs-15/tests/pageload-tracing.test.ts

@@ -1,7 +1,7 @@
 import { expect, test } from '@playwright/test';
 import { waitForTransaction } from '@sentry-internal/test-utils';
 
-test('all server component transactions should be attached to the pageload request span', async ({ page }) => {
+test('App router transactions should be attached to the pageload request span', async ({ page }) => {
   const serverTransactionPromise = waitForTransaction('nextjs-15', async transactionEvent => {
     return transactionEvent?.transaction === 'GET /pageload-tracing';
   });

packages/nextjs/src/server/index.ts

@@ -121,8 +121,18 @@ export function init(options: NodeOptions): NodeClient | undefined {
   applySdkMetadata(opts, 'nextjs', ['nextjs', 'node']);
 
   const client = nodeInit(opts);
-
+  // If we encounter a span emitted by Next.js, we do not want to sample it
+  // The reason for this is that the data quality of the spans varies, it is different per version of Next,
+  // and we need to keep our manual instrumentation around for the edge runtime anyhow.
+  // BUT we only do this if we don't have a parent span with a sampling decision yet (or if the parent is remote)
   client?.on('beforeSampling', ({ spanAttributes, spanName, parentSampled, parentContext }, samplingDecision) => {
+    // We "whitelist" the "BaseServer.handleRequest" span, since that one is responsible for App Router requests, which are actually useful for us.
+    // HOWEVER, that span is not only responsible for App Router requests, which is why we additionally filter for certain transactions in an
+    // event processor further below.
+    if (spanAttributes['next.span_type'] === 'BaseServer.handleRequest') {
+      return;
+    }
+
     if (
       (spanAttributes['next.span_type'] || NEXTJS_SPAN_NAME_PREFIXES.some(prefix => spanName.startsWith(prefix))) &&
       (parentSampled === undefined || parentContext?.isRemote)
@@ -144,7 +154,8 @@ export function init(options: NodeOptions): NodeClient | undefined {
 
           // We only want to use our HTTP integration/instrumentation for app router requests, which are marked with the `sentry.rsc` attribute.
           if (
-            event.contexts?.trace?.data?.[SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN] === 'auto.http.otel.http' &&
+            (event.contexts?.trace?.data?.[SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN] === 'auto.http.otel.http' ||
+              event.contexts?.trace?.data?.['next.span_type'] === 'BaseServer.handleRequest') &&
             event.contexts?.trace?.data?.['sentry.rsc'] !== true
           ) {
             return null;