test: Check that runtimes drop the server spans of managed tunnel requests by logaretm · Pull Request #25414 · getsentry/sentry-javascript

logaretm · 2026-10-15T15:33:45Z

Runtimes only translate tunnel: true while meta-frameworks answer the requests, so nothing pinned down that their server spans get dropped.

This PR asserts that runtime SDKs level (Cloudflare, Node, Bun and Deno) ignores the tunneled span so we don't have to chase it around. This is one benefit of having tunnels as a core SDK capability, this PR just tests that.

closes #25462

github-actions · 2026-10-15T15:44:41Z

size-limit report 📦

Path	Size	% Change	Change
@sentry/browser	31.03 kB	+3.87%	+1.16 kB 🔺
@sentry/browser - with treeshaking flags	28.82 kB	+2.87%	+803 B 🔺
@sentry/browser - with treeshaking flags tracing without tracing	28.72 kB	+2.86%	+798 B 🔺
@sentry/browser (incl. Tracing)	53.24 kB	+2.26%	+1.17 kB 🔺
@sentry/browser (incl. Tracing + Span Streaming)	53.24 kB	+2.24%	+1.17 kB 🔺
@sentry/browser (incl. Tracing, Profiling)	56.24 kB	+2.24%	+1.23 kB 🔺
@sentry/browser (incl. Tracing, Replay)	92.99 kB	+1.28%	+1.17 kB 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags	81.48 kB	+1%	+799 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas)	97.74 kB	+1.24%	+1.19 kB 🔺
@sentry/browser (incl. Tracing, Replay, Feedback)	110.67 kB	+1.09%	+1.19 kB 🔺
@sentry/browser (incl. Feedback)	48.6 kB	+2.55%	+1.21 kB 🔺
@sentry/browser (incl. sendFeedback)	36.12 kB	+3.37%	+1.18 kB 🔺
@sentry/browser (incl. FeedbackAsync)	41.25 kB	+3%	+1.2 kB 🔺
@sentry/browser (incl. Metrics)	32.03 kB	+3.65%	+1.13 kB 🔺
@sentry/browser (incl. Logs)	32.32 kB	+3.65%	+1.14 kB 🔺
@sentry/browser (incl. Metrics & Logs)	33.01 kB	+3.78%	+1.2 kB 🔺
@sentry/react	32.84 kB	+3.58%	+1.13 kB 🔺
@sentry/react (incl. Tracing)	55.52 kB	+2.08%	+1.13 kB 🔺
@sentry/vue	39.21 kB	+3.09%	+1.17 kB 🔺
@sentry/vue (incl. Tracing)	56.25 kB	+2.26%	+1.24 kB 🔺
@sentry/svelte	31.06 kB	+3.88%	+1.16 kB 🔺
@sentry/remix (Remix 3 client bundle)	58.28 kB	+2.11%	+1.2 kB 🔺
CDN Bundle	32.36 kB	+2.27%	+716 B 🔺
CDN Bundle (incl. Tracing)	53.3 kB	+1.38%	+723 B 🔺
CDN Bundle (incl. Logs, Metrics)	34.55 kB	+2.13%	+718 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics)	55.25 kB	+1.34%	+728 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics)	75.52 kB	+1.01%	+751 B 🔺
CDN Bundle (incl. Tracing, Replay)	90.97 kB	+0.8%	+717 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics)	92.96 kB	+0.86%	+785 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback)	97.14 kB	+0.77%	+737 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics)	99.14 kB	+0.81%	+792 B 🔺
CDN Bundle - uncompressed	95.04 kB	+2%	+1.86 kB 🔺
CDN Bundle (incl. Tracing) - uncompressed	157.88 kB	+1.19%	+1.85 kB 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed	101.58 kB	+1.87%	+1.86 kB 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed	163.84 kB	+1.15%	+1.85 kB 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed	231.62 kB	+0.82%	+1.86 kB 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed	278.05 kB	+0.67%	+1.85 kB 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed	283.99 kB	+0.66%	+1.85 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed	291.75 kB	+0.64%	+1.85 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed	297.69 kB	+0.63%	+1.85 kB 🔺
@sentry/nextjs (client)	57.92 kB	+1.98%	+1.12 kB 🔺
@sentry/sveltekit (client)	53.58 kB	+2.17%	+1.14 kB 🔺
@sentry/core/server	41.85 kB	+1.78%	+730 B 🔺
@sentry/core/browser	13.81 kB	-	-
@sentry/node	153.84 kB	+0.5%	+757 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection)	84.04 kB	-	-
@sentry/node - without tracing	95.58 kB	+0.75%	+707 B 🔺
@sentry/node - without channel injection	131.89 kB	+0.58%	+748 B 🔺
@sentry/aws-serverless	103.84 kB	+0.79%	+805 B 🔺
@sentry/cloudflare (withSentry) - minified	212.53 kB	+0.92%	+1.93 kB 🔺
@sentry/cloudflare (withSentry)	526.33 kB	+0.73%	+3.8 kB 🔺
@sentry/nextjs/cloudflare (withSentry) - minified	230.21 kB	+0.85%	+1.93 kB 🔺

View base workflow run

logaretm · 2026-10-15T21:56:02Z

bugbot run

cursor

Stale Bugbot comment from a previous run.

logaretm · 2026-10-16T02:14:26Z

bugbot run

logaretm · 2026-10-16T02:14:44Z

bugbot run

cursor

✅ Bugbot reviewed your changes and found no new issues!

1 issue from previous review remains unresolved.

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit a729215. Configure here.}

…uests Meta-framework SDKs answer the tunnel requests. Cloudflare, Node, Bun and Deno only translate `tunnel: true` so the core client drops the server span they create for them, which these tests pin down.

logaretm added this pull request to stack #25201 October 15, 2026 15:33

logaretm force-pushed the awad/tunnel-cloudflare branch from 323db37 to 776be29 Compare October 15, 2026 15:36

logaretm force-pushed the awad/tunnel-cloudflare branch from 776be29 to e0f36f5 Compare October 15, 2026 15:51

logaretm force-pushed the awad/tunnel-cloudflare branch from 4a248fa to 37afe7c Compare October 15, 2026 16:38

logaretm force-pushed the awad/tunnel-cloudflare branch from 37afe7c to 03d8225 Compare October 15, 2026 17:15

logaretm force-pushed the awad/tunnel-cloudflare branch from 03d8225 to 3e455f4 Compare October 15, 2026 17:36

logaretm changed the title ~~feat(cloudflare): Serve the managed tunnel from the request wrapper~~ test: Check that runtimes drop the server spans of managed tunnel requests Oct 15, 2026

logaretm force-pushed the awad/tunnel-cloudflare branch from 3e455f4 to fa2d7a6 Compare October 15, 2026 17:44

logaretm force-pushed the awad/tunnel-cloudflare branch from fa2d7a6 to 034ec1f Compare October 15, 2026 18:16

logaretm force-pushed the awad/tunnel-cloudflare branch from 034ec1f to b254fc1 Compare October 15, 2026 18:26

logaretm force-pushed the awad/tunnel-cloudflare branch from b254fc1 to b8540c7 Compare October 15, 2026 18:28

cursor Bot reviewed Oct 15, 2026

View reviewed changes

Comment thread packages/cloudflare/test/request.test.ts

Comment thread dev-packages/node-integration-tests/suites/tracing/managed-tunnel/test.ts Outdated

Comment thread dev-packages/node-integration-tests/suites/tracing/managed-tunnel/test.ts

logaretm force-pushed the awad/tunnel-cloudflare branch from b8540c7 to a729215 Compare October 16, 2026 01:51

cursor Bot reviewed Oct 16, 2026

View reviewed changes

logaretm force-pushed the awad/tunnel-cloudflare branch from a729215 to bcba572 Compare October 16, 2026 02:25

logaretm force-pushed the awad/tunnel-cloudflare branch from bcba572 to c64e39d Compare October 16, 2026 04:23

logaretm force-pushed the awad/tunnel-cloudflare branch from c64e39d to af457ac Compare October 16, 2026 14:00

logaretm force-pushed the awad/tunnel-cloudflare branch from af457ac to 1726668 Compare October 16, 2026 14:37

logaretm force-pushed the awad/tunnel-cloudflare branch from 1726668 to 353179b Compare October 16, 2026 16:15

logaretm mentioned this pull request Oct 16, 2026

Cloudflare managed tunnel #25462

Open

logaretm force-pushed the awad/tunnel-cloudflare branch from 353179b to 6732028 Compare October 16, 2026 17:56

logaretm force-pushed the awad/tunnel-cloudflare branch from 6732028 to 3f85c31 Compare October 16, 2026 20:56

logaretm added 3 commits October 16, 2026 17:01

test: Check that runtimes drop the server spans of managed tunnel req…

44036c9

…uests Meta-framework SDKs answer the tunnel requests. Cloudflare, Node, Bun and Deno only translate `tunnel: true` so the core client drops the server span they create for them, which these tests pin down.

test(bun): Run the managed tunnel suite only with the Bun SDK

d8b4eff

test(node): Assert the tunnel request sends no server span at all

6fb8fd0

logaretm force-pushed the awad/tunnel-cloudflare branch from 3f85c31 to 6fb8fd0 Compare October 16, 2026 21:02

Navigation Menu

Repository navigation

test: Check that runtimes drop the server spans of managed tunnel requests - #25414

test: Check that runtimes drop the server spans of managed tunnel requests#25414
logaretm wants to merge 3 commits into
awad/tunnel-tanstackstartfrom
awad/tunnel-cloudflare

logaretm commented Oct 15, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Oct 15, 2026 •

edited

Loading

Uh oh!

logaretm commented Oct 15, 2026

Uh oh!

cursor Bot left a comment •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

logaretm commented Oct 16, 2026

Uh oh!

logaretm commented Oct 16, 2026

Uh oh!

cursor Bot left a comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

logaretm commented Oct 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions Bot commented Oct 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

size-limit report 📦

Uh oh!

logaretm commented Oct 15, 2026

Uh oh!

cursor Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

logaretm commented Oct 16, 2026

Uh oh!

logaretm commented Oct 16, 2026

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

logaretm commented Oct 15, 2026 •

edited

Loading

github-actions Bot commented Oct 15, 2026 •

edited

Loading

cursor Bot left a comment •

edited

Loading