Skip to content

fix(minidump): Update minidumper-child to skip macOS system dialog on successful dumping - #1368

Open
timfish wants to merge 2 commits into
getsentry:masterfrom
timfish:feat/update-minidumper-child
Open

timfish wants to merge 2 commits into
getsentry:masterfrom
timfish:feat/update-minidumper-child

Conversation

@timfish

@timfish timfish commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

On macOS, the system crash handler was displayed even when we successfully capture and send the minidump. This is in contrast to the Windows behaviour where WER is skipped when successful. This change brings the macOS behaviour in line with Windows, Breakpad and Crashpad.

@timfish
timfish marked this pull request as ready for review October 22, 2026 11:51
@timfish
timfish requested a review from a team as a code owner October 22, 2026 11:51
@codecov

codecov Bot commented Oct 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.33%. Comparing base (a57b91c) to head (9cb04b6).
⚠️ Report is 192 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1368      +/-   ##
==========================================
+ Coverage   73.81%   74.33%   +0.52%     
==========================================
  Files          64       79      +15     
  Lines        7538    10044    +2506     
==========================================
+ Hits         5564     7466    +1902     
- Misses       1974     2578     +604     

This branch has not been deployed

No deployments
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.

2 participants