New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
meta(changelog): Update changelog for 7.52.0 #8118
Conversation
[Gitflow] Merge master into develop
This fixes a helper function used in our implementation of `ignoreErrors` so that in the case of linked errors, it filters on the primary error (the one directly caught by `captureException`) rather than the error in the primary error's `cause` property[1]. See #8079 for screenshots and a more detailed explanation. Fixes #8079. [1] https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Error/cause
Co-authored-by: Abhijeet Prasad <aprasad@sentry.io>
As mentioned by @ryan953 [here](https://github.com/getsentry/getsentry/pull/10418/files#r1183010062), we should check relative URLs properly as well for capturing network details.
…me-origin urls (#8114) We need to adjust our `tracePropagationTargets` default regex to account for protocol-relative URLs. These were previously classified as same-origin, relative URLs, causing tracing headers to be attached which in turn potentially caused CORS errors for users.
CHANGELOG.md
Outdated
- fix(tracing-internal): Avoid classifying protocol-relative URLs as same-origin urls (#8114) | ||
- ref: Hoist `createCheckinEnvelope` to core package (#8082) | ||
|
||
**Important Next.js SDK changes:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's just stick with the pattern we've used before? Also this section should go above the individual items above, like what we did with 7.51.0
.
**Important Next.js SDK changes:** | |
### Important Changes |
CHANGELOG.md
Outdated
- fix(docs): Change to `addTracingExtensions` was not documented in MIGRATION.md (#8101) | ||
- fix(replay): Check relative URLs correctly (#8024) | ||
- fix(tracing-internal): Avoid classifying protocol-relative URLs as same-origin urls (#8114) | ||
- ref: Hoist `createCheckinEnvelope` to core package (#8082) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this need to be in the changelog?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather have this than not
CHANGELOG.md
Outdated
- fix(tracing-internal): Avoid classifying protocol-relative URLs as same-origin urls (#8114) | ||
- ref: Hoist `createCheckinEnvelope` to core package (#8082) | ||
|
||
**Important Next.js SDK changes:** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we move this section to the top and have the other changes below? This way we're consistent with the changelog messages from the past releases.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure
- feat(nextjs): Add API method to wrap API routes with crons instrumentation (#8084) | ||
- feat(nextjs): Add automatic monitors for Vercel Cron Jobs (#8088) | ||
|
||
### Other changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Other changes | |
### Additional Features and Fixes |
size-limit report 📦
|
No description provided.