Skip to content

fix(tanstackstart-react): Drop server transactions for tunnel route requests - #21769

Merged
logaretm merged 3 commits into
developfrom
fix/tanstackstart-ignore-tunnel-route-transactions
Jun 26, 2026
Merged

fix(tanstackstart-react): Drop server transactions for tunnel route requests#21769
logaretm merged 3 commits into
developfrom
fix/tanstackstart-ignore-tunnel-route-transactions

Conversation

@logaretm

@logaretmlogaretm commented Jun 24, 2026

Copy link
Copy Markdown
Member

The server SDK was turning every incoming tunnel route request into an http.server transaction. This drops tunnel traffic for both the static and streamed (traceLifecycle: 'stream') span lifecycles.

closes#21555

@logaretm
logaretm requested a review from a team as a code ownerJune 24, 2026 21:27
@logaretm
logaretm requested review from chargome, mydea, nicohrubec and s1gr1d and removed request for a team and chargomeJune 24, 2026 21:27
return {
handlers: {
POST: async ({ request }) => {
const activeSpan = getActiveSpan();

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

any way to maybe solve this via suppressTracing instead? 🤔

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

The span would be created outside of this callback so it wouldn't work

Comment threadpackages/tanstackstart-react/src/vite/tunnelRoute.ts Outdated
Comment threadpackages/tanstackstart-react/src/server/sdk.ts Outdated
Comment threadpackages/tanstackstart-react/src/vite/tunnelRoute.ts
@logaretm
logaretm requested review from Lms24 and nicohrubecJune 25, 2026 14:44
@logaretm
logaretm merged commit 54e995d into developJun 26, 2026
554 of 558 checks passed
@logaretm
logaretm deleted the fix/tanstackstart-ignore-tunnel-route-transactions branch June 26, 2026 15:03
Sign up for freeto 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.

TanStack Start Tunnel not being ignored by the SDK

3 participants

@logaretm@mydea@nicohrubec