Skip to content

feat(remix): Serve the managed tunnel from the Remix 3 router middleware - #25420

Draft
logaretm wants to merge 2 commits into
awad/tunnel-astrofrom
awad/tunnel-remix
Draft

logaretm wants to merge 2 commits into
awad/tunnel-astrofrom
awad/tunnel-remix

Conversation

@logaretm

@logaretm logaretm commented Oct 15, 2026 •

Copy link
Copy Markdown
Member

Remix 3 apps had no way to serve tunnel: true, fixed by answering tunnel requests in the Sentry router middleware, which already runs before route matching and the app's own middleware.

The e2e tunnel variant checks the tunnel answers while an auth middleware rejects every other POST.

@logaretm
logaretm added this pull request to stack #25201 October 15, 2026 16:13
@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 30.7 kB +2.76% +824 B 🔺
@sentry/browser - with treeshaking flags 28.84 kB +2.95% +825 B 🔺
@sentry/browser - with treeshaking flags tracing without tracing 28.73 kB +2.92% +814 B 🔺
@sentry/browser (incl. Tracing) 52.88 kB +1.55% +806 B 🔺
@sentry/browser (incl. Tracing + Span Streaming) 52.88 kB +1.56% +811 B 🔺
@sentry/browser (incl. Tracing, Profiling) 55.8 kB +1.44% +788 B 🔺
@sentry/browser (incl. Tracing, Replay) 92.61 kB +0.87% +793 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 81.47 kB +0.98% +785 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 97.34 kB +0.83% +794 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 110.31 kB +0.76% +826 B 🔺
@sentry/browser (incl. Feedback) 48.22 kB +1.75% +828 B 🔺
@sentry/browser (incl. sendFeedback) 35.78 kB +2.41% +839 B 🔺
@sentry/browser (incl. FeedbackAsync) 40.87 kB +2.06% +822 B 🔺
@sentry/browser (incl. Metrics) 31.69 kB +2.56% +789 B 🔺
@sentry/browser (incl. Logs) 31.97 kB +2.55% +794 B 🔺
@sentry/browser (incl. Metrics & Logs) 32.66 kB +2.69% +854 B 🔺
@sentry/react 32.5 kB +2.53% +800 B 🔺
@sentry/react (incl. Tracing) 55.19 kB +1.48% +803 B 🔺
@sentry/vue 38.84 kB +2.11% +802 B 🔺
@sentry/vue (incl. Tracing) 55.81 kB +1.46% +803 B 🔺
@sentry/svelte 30.73 kB +2.78% +829 B 🔺
@sentry/remix (Remix 3 client bundle) 57.94 kB +1.51% +857 B 🔺
CDN Bundle 32.13 kB +1.55% +488 B 🔺
CDN Bundle (incl. Tracing) 53.09 kB +0.96% +504 B 🔺
CDN Bundle (incl. Logs, Metrics) 34.34 kB +1.49% +502 B 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) 55.03 kB +0.93% +507 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 75.5 kB +0.98% +731 B 🔺
CDN Bundle (incl. Tracing, Replay) 90.97 kB +0.81% +725 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 92.96 kB +0.86% +785 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 97.1 kB +0.73% +698 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 99.13 kB +0.81% +788 B 🔺
CDN Bundle - uncompressed 94.4 kB +1.32% +1.22 kB 🔺
CDN Bundle (incl. Tracing) - uncompressed 157.24 kB +0.78% +1.21 kB 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed 100.94 kB +1.23% +1.22 kB 🔺
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 163.2 kB +0.75% +1.21 kB 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 231.31 kB +0.68% +1.56 kB 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 277.74 kB +0.56% +1.54 kB 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 283.69 kB +0.55% +1.54 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 291.44 kB +0.54% +1.54 kB 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 297.38 kB +0.53% +1.54 kB 🔺
@sentry/nextjs (client) 57.59 kB +1.4% +791 B 🔺
@sentry/sveltekit (client) 53.3 kB +1.63% +854 B 🔺
@sentry/core/server 41.78 kB +1.61% +660 B 🔺
@sentry/core/browser 13.81 kB - -
@sentry/node 153.15 kB +0.46% +687 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 83.81 kB - -
@sentry/node - without tracing 95.29 kB +0.72% +676 B 🔺
@sentry/node - without channel injection 131.35 kB +0.51% +664 B 🔺
@sentry/aws-serverless 103.48 kB +0.69% +708 B 🔺
@sentry/cloudflare (withSentry) - minified 212.31 kB +0.82% +1.71 kB 🔺
@sentry/cloudflare (withSentry) 525.93 kB +0.66% +3.4 kB 🔺
@sentry/nextjs/cloudflare (withSentry) - minified 229.98 kB +0.75% +1.71 kB 🔺

View base workflow run

@logaretm
logaretm force-pushed the awad/tunnel-remix branch 2 times, most recently from 769b893 to 3bb11c2 Compare October 15, 2026 18:16
@logaretm
logaretm force-pushed the awad/tunnel-remix branch 2 times, most recently from 2a02a85 to d8aa95d Compare October 15, 2026 18:28
@logaretm

Copy link
Copy Markdown
Member Author

bugbot run

@cursor cursor Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stale Bugbot comment from a previous run.

Comment thread dev-packages/e2e-tests/test-applications/remix-v3/tests/tunnel.test.ts Outdated
@logaretm

Copy link
Copy Markdown
Member Author

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

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

Reviewed by Cursor Bugbot for commit 4f1d836. Configure here.

Sentry's middleware is already prepended to every `createRouter`, so it now
answers tunnel requests before route matching and the app's own middleware.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant