Skip to content

ref(minidump): Instrument additional exceptions#6138

Open
jjbayer wants to merge 5 commits into
masterfrom
ref/instrument-multi-exception
Open

ref(minidump): Instrument additional exceptions#6138
jjbayer wants to merge 5 commits into
masterfrom
ref/instrument-multi-exception

Conversation

@jjbayer

@jjbayer jjbayer commented Jun 25, 2026

Copy link
Copy Markdown
Member

Additional exceptions in a minidump event should work with javascript but not with additional native exceptions, because sentry will send at most one native symbolication request to symbolicator.

Count occurrences of multiple exceptions so we know how commonly this happens.

ref: #6046

@jjbayer jjbayer changed the title Ref/instrument multi exception ref(minidump): Instrument additional exceptions Jun 25, 2026
@jjbayer jjbayer marked this pull request as ready for review June 26, 2026 08:54
@jjbayer jjbayer requested a review from a team as a code owner June 26, 2026 08:54
@jjbayer jjbayer enabled auto-merge June 26, 2026 08:55
Comment thread relay-server/src/utils/native.rs
Comment thread relay-server/src/utils/native.rs Outdated

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 49e71dc. Configure here.

Comment thread relay-server/src/utils/native.rs
Comment thread relay-server/src/utils/native.rs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant