Skip to content

meta: Sync develop -> master #7036

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

Merged
merged 12 commits into from
Feb 2, 2023
Merged

meta: Sync develop -> master #7036

merged 12 commits into from
Feb 2, 2023

Conversation

Lms24
Copy link
Member

@Lms24 Lms24 commented Feb 2, 2023

manual sync while waiting on #7034

mydea and others added 12 commits February 1, 2023 11:45
[Gitflow] Merge master into develop
Otherwise, workflows will not be triggered due to security issues.
See peter-evans/create-pull-request#48 for details.
We want to entirely stop recording the replay once we receive a 429 rate limit response. This patch gets rid of all the "pause on rate limit" logic. Because we already stop when we receive other http error responses, we can simply use this logic instead.
This was unintentionally find/replaced. Our Network tab basically had no data (although some did, and Im not sure how thats possible).
This was unintentionally removed in #6645 - it is still deprecated and will be removed at a later date.

Co-authored-by: Lukas Stracke <[email protected]>
…7011)

Report all integrations from BaseClient, even the ones that were added after the SDK intialization. This patch changes reading the list of integrations from `options.integrations` to `client._integrations`.
Add rrweb's `_cssText` property to the list of reserved properties to be excluded from mangling.
@Lms24 Lms24 requested a review from mydea February 2, 2023 09:36
Copy link
Member

@mydea mydea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't forget to merge with merge commit ;)

@Lms24 Lms24 merged commit 503a1c2 into master Feb 2, 2023
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.

4 participants