Skip to content

Conversation

@andreiborza
Copy link
Member

No description provided.

chargome and others added 6 commits May 19, 2025 15:19
[Gitflow] Merge master into develop
This PR adds the external contributor to the CHANGELOG.md file, so that
they are credited for their contribution. See #14629

Co-authored-by: mydea <[email protected]>
It was noticed that we were creating breadcrumbs for sentry-internal
requests, which is not desired. This was introduced by us moving
breadcrumb generation out of the base http/fetch instrumentation into
our own, where we did not look at `suppressTracing`. Now, when tracing
is suppressed, no fetch/http breadcrumbs will be created.
Resolves: #16309 

I tried to make as few modifications as possible for easier updates from
the original implementation in the future.

- Converted `require`s and `module.export`s to `import`s and `exports`
- Created a local `eslint` configuration
- Hard-coded `name` and `version` to the vendored version instead of
reading from `package.json`
@andreiborza andreiborza requested review from Lms24, chargome and mydea May 20, 2025 10:23
… entries (#16341)

Resolves: #16298 

Also exported `ErrorBoundary` and `browserTracingIntegration` from
`cloudflare` entry.
@github-actions
Copy link
Contributor

github-actions bot commented May 20, 2025

size-limit report 📦

Path Size % Change Change
@sentry/browser 24.04 kB added added
@sentry/browser - with treeshaking flags 23.7 kB added added
@sentry/browser (incl. Tracing) 38.46 kB added added
@sentry/browser (incl. Tracing, Replay) 76.58 kB added added
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 69.61 kB added added
@sentry/browser (incl. Tracing, Replay with Canvas) 81.35 kB added added
@sentry/browser (incl. Tracing, Replay, Feedback) 93.46 kB added added
@sentry/browser (incl. Feedback) 40.83 kB added added
@sentry/browser (incl. sendFeedback) 28.79 kB added added
@sentry/browser (incl. FeedbackAsync) 33.66 kB added added
@sentry/react 25.82 kB added added
@sentry/react (incl. Tracing) 40.43 kB added added
@sentry/vue 28.44 kB added added
@sentry/vue (incl. Tracing) 40.28 kB added added
@sentry/svelte 24.07 kB added added
CDN Bundle 25.24 kB added added
CDN Bundle (incl. Tracing) 38.53 kB added added
CDN Bundle (incl. Tracing, Replay) 74.31 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) 79.76 kB added added
CDN Bundle - uncompressed 73.63 kB added added
CDN Bundle (incl. Tracing) - uncompressed 114.03 kB added added
CDN Bundle (incl. Tracing, Replay) - uncompressed 228 kB added added
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 240.82 kB added added
@sentry/nextjs (client) 42.08 kB added added
@sentry/sveltekit (client) 38.95 kB added added
@sentry/node 149.47 kB added added
@sentry/node - without tracing 98.1 kB added added
@sentry/aws-serverless 123.43 kB added added

mydea and others added 2 commits May 20, 2025 13:03
Just a small refactor, noticed this while working on other browser sdk
stuff. Moving this into a dedicated file decouples this from the core
SDK functionality, which this IMHO is not (anymore). Also, we can
streamline the code slightly for efficiency.
@andreiborza andreiborza force-pushed the prepare-release/9.21.0 branch from 605e4e4 to 8902822 Compare May 20, 2025 11:25
@andreiborza andreiborza merged commit b8916bb into master May 20, 2025
155 checks passed
@andreiborza andreiborza deleted the prepare-release/9.21.0 branch May 20, 2025 11:39
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.

8 participants