Skip to content

feat(browser): Add outgoingRequest context to fetch errors #15850

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

mydea
Copy link
Member

@mydea mydea commented Mar 27, 2025

For now, this only adds method and url fields.

Implements #15846 for fetch only. This is probably the most important, not sure if we also want to add this to XMLHttpRequest handling (it is possible there but a bit trickier, as we'd need to parse method/url out of the request etc)

@mydea mydea requested a review from lforst March 27, 2025 11:23
@mydea mydea self-assigned this Mar 27, 2025
For now, this only adds `method` and `url` fields.
@mydea mydea force-pushed the fn/fetch-error-context branch from 53e3ea4 to c4f3edf Compare March 27, 2025 11:25
Copy link
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 23.14 KB +0.29% +67 B 🔺
@sentry/browser - with treeshaking flags 22.95 KB +0.32% +73 B 🔺
@sentry/browser (incl. Tracing) 36.55 KB +0.17% +62 B 🔺
@sentry/browser (incl. Tracing, Replay) 73.73 KB +0.11% +81 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 67.08 KB +0.12% +79 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 78.38 KB +0.1% +80 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 90.96 KB +0.1% +89 B 🔺
@sentry/browser (incl. Feedback) 40.28 KB +0.19% +75 B 🔺
@sentry/browser (incl. sendFeedback) 27.79 KB +0.28% +79 B 🔺
@sentry/browser (incl. FeedbackAsync) 32.57 KB +0.23% +74 B 🔺
@sentry/react 24.94 KB +0.31% +78 B 🔺
@sentry/react (incl. Tracing) 38.44 KB +0.14% +54 B 🔺
@sentry/vue 27.36 KB +0.23% +62 B 🔺
@sentry/vue (incl. Tracing) 38.25 KB +0.18% +69 B 🔺
@sentry/svelte 23.17 KB +0.26% +60 B 🔺
CDN Bundle 24.4 KB +0.29% +72 B 🔺
CDN Bundle (incl. Tracing) 36.56 KB +0.14% +51 B 🔺
CDN Bundle (incl. Tracing, Replay) 71.58 KB +0.08% +54 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 76.76 KB +0.08% +58 B 🔺
CDN Bundle - uncompressed 71.16 KB +0.33% +239 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 108.33 KB +0.2% +219 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 219.62 KB +0.1% +218 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 232.18 KB +0.1% +218 B 🔺
@sentry/nextjs (client) 39.72 KB +0.13% +51 B 🔺
@sentry/sveltekit (client) 36.97 KB +0.15% +55 B 🔺
@sentry/node 142.93 KB +0.02% +29 B 🔺
@sentry/node - without tracing 96.14 KB +0.04% +33 B 🔺
@sentry/aws-serverless 120.48 KB +0.03% +29 B 🔺

View base workflow run

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant