-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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: Update CHANGELOG for 8.1.0 #12062
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
[Gitflow] Merge master into develop
Just to ensure we do not regress here. Request isolation should work even without tracing (which means, even without the express integration added - that is only added when tracing is enabled, today!). Really, the only thing missing should be `transactionName`.
Just a test showing how stuff behaves if: 1. `init` is called without DSN 2. `httpIntegration` is manually added 3. `init` is called again, later, with DSN --------- Co-authored-by: Andrei <168741329+andreiborza@users.noreply.github.com>
fix(aws-serverless): Add `awslambda-auto` to `package.json` exports
Just noticed that there's not really a reason to have the two separate warnings. Let's save some bytes.
We had this as TBD. I checked with @ryan953 and put the version that we want in, plus a disclaimer.
…tions are not available (#12026)
…2017) The prior fix was incomplete, because we were still using `getDefaultIntegrations()` inside of node's `init`, so the deps where pulled in anyhow. Furthermore, it seems that `preserveModules: false` for `@sentry/node` also prevented this from working as expected. So this PR does two things: 1. Set `preserveModules: true` so that tree-shaking can work as expected (😿 ) 2. Expose a new `initWithoutDefaultIntegrations` method from `@sentry/node` which AWS uses, which avoids including any integrations by default. You have to pass your own `defaultIntegrations` to it. This is not the prettiest solution, but I couldn't think of anything much better 😬 I also added a size-limit entry to keep track of this.
…2022) Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
Extracted this out from #12020 For whatever reason, this does not seemed to play well with how we run the proxy server via ts-node. Looking into this more, this is actually not really necessary at all, so I just stripped this out in favor of just running the proxy as a `.mjs` file directly - works just as well!
Supersedes #11912 I just cleaned up and renamed this to align with other things
Lms24
requested review from
a team,
mydea,
stephanie-anderson and
s1gr1d
and removed request for
a team and
stephanie-anderson
May 16, 2024 07:48
mydea
approved these changes
May 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.