Skip to content

feat(browser)!: Use http.client span op for streamed fetch bodies - #22684

Open
msonnb wants to merge 2 commits into
developfrom
ms/js-3105-fetch-stream-op
Open

feat(browser)!: Use http.client span op for streamed fetch bodies#22684
msonnb wants to merge 2 commits into
developfrom
ms/js-3105-fetch-stream-op

Conversation

@msonnb

@msonnb msonnb commented Jul 27, 2026

Copy link
Copy Markdown
Member

Use http.client instead of http.client.stream for the sibling span fetchStreamPerformanceIntegration creates. Streaming is preserved in the new http.response.body.streaming attribute.

This span is a sibling of the regular request span and already shared its name, so it now shares the op too. The test span finders key on the streaming attribute to tell the two apart.

Part of #22446

@linear-code

linear-code Bot commented Jul 27, 2026

Copy link
Copy Markdown

JS-3105

@github-actions

github-actions Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.84 kB - -
@sentry/browser - with treeshaking flags 28.05 kB - -
@sentry/browser (incl. Tracing) 47.14 kB +0.04% +18 B 🔺
@sentry/browser (incl. Tracing + Span Streaming) 47.13 kB +0.02% +6 B 🔺
@sentry/browser (incl. Tracing, Profiling) 51.85 kB +0.05% +23 B 🔺
@sentry/browser (incl. Tracing, Replay) 86.45 kB +0.03% +20 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.89 kB +0.04% +29 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 91.17 kB +0.03% +19 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 103.8 kB +0.01% +6 B 🔺
@sentry/browser (incl. Feedback) 47.15 kB - -
@sentry/browser (incl. sendFeedback) 34.68 kB - -
@sentry/browser (incl. FeedbackAsync) 39.78 kB - -
@sentry/browser (incl. Metrics) 30.91 kB - -
@sentry/browser (incl. Logs) 31.14 kB - -
@sentry/browser (incl. Metrics & Logs) 31.82 kB - -
@sentry/react 31.63 kB - -
@sentry/react (incl. Tracing) 49.37 kB +0.04% +17 B 🔺
@sentry/vue 34.76 kB - -
@sentry/vue (incl. Tracing) 49.09 kB +0.03% +14 B 🔺
@sentry/svelte 29.87 kB - -
CDN Bundle 31.89 kB - -
CDN Bundle (incl. Tracing) 47.48 kB +0.04% +15 B 🔺
CDN Bundle (incl. Logs, Metrics) 33.44 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 48.85 kB +0.02% +9 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) 72.8 kB - -
CDN Bundle (incl. Tracing, Replay) 85.12 kB +0.02% +14 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.42 kB +0.03% +21 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 90.87 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.19 kB +0.01% +9 B 🔺
CDN Bundle - uncompressed 95.11 kB - -
CDN Bundle (incl. Tracing) - uncompressed 142.37 kB +0.03% +42 B 🔺
CDN Bundle (incl. Logs, Metrics) - uncompressed 99.82 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 146.35 kB +0.03% +42 B 🔺
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 224.58 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 261.63 kB +0.02% +42 B 🔺
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 265.59 kB +0.02% +42 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 275.33 kB +0.02% +42 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 279.29 kB +0.02% +42 B 🔺
@sentry/nextjs (client) 51.96 kB +0.03% +14 B 🔺
@sentry/sveltekit (client) 47.55 kB +0.04% +18 B 🔺
@sentry/core/server 79.78 kB - -
@sentry/core/browser 51.61 kB - -
@sentry/node 121.06 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 166 B - -
@sentry/node - without tracing 84.48 kB +0.01% +1 B 🔺
@sentry/aws-serverless 92.7 kB - -
@sentry/cloudflare (withSentry) - minified 197.8 kB - -
@sentry/cloudflare (withSentry) 486.19 kB - -

View base workflow run

@msonnb
msonnb force-pushed the ms/js-3105-aws-lambda-op branch from 4a318cf to 5702f0c Compare July 27, 2026 12:10
@msonnb
msonnb force-pushed the ms/js-3105-fetch-stream-op branch from 8076339 to df12758 Compare July 27, 2026 12:10
@msonnb
msonnb changed the base branch from ms/js-3105-aws-lambda-op to develop July 27, 2026 13:08
@msonnb
msonnb force-pushed the ms/js-3105-fetch-stream-op branch from df12758 to dc82459 Compare July 27, 2026 13:51
Streaming changes response behaviour, not the operation category. The
sibling span is now marked with `http.response.body.streaming` instead
of an `http.client.stream` op.

Ref: JS-3105

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@msonnb
msonnb force-pushed the ms/js-3105-fetch-stream-op branch from dc82459 to 2bcb80c Compare July 29, 2026 08:16
@msonnb
msonnb marked this pull request as ready for review July 29, 2026 08:49
@msonnb
msonnb requested a review from a team as a code owner July 29, 2026 08:49
@msonnb
msonnb requested review from Lms24 and logaretm and removed request for a team July 29, 2026 08:49
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