-
Notifications
You must be signed in to change notification settings - Fork 16.4k
fix: crashReporter incompatible with sandbox on Linux (#23265) #23461
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
Conversation
Release Notes Persisted
|
Does |
@vyunikov on Linux, it does not. I don't know what use case you are talking about. To initialize the crash reporter in a Node child process on Linux, you'll need to use |
Ref #25579 |
@nornagon Thanks for your response. Sorry, I was probably a bit unclear. |
|
@nornagon I am using Electron 9.2.0 and it doesn't look like |
@vyunikov perhaps you could open a new issue with a small repro illustrating the issue you're having? |
Will do, thanks |
Backport of #23265.
Notes:
crashReporter
is now explicitly initialized only in the main process, and implicitly initialized in other child processes. This fixes an issue preventing the crash reporter from functioning in sandboxed renderers on Linux.