Skip to content
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

Report bugs in a separate process #9083

Merged
merged 1 commit into from Apr 19, 2021

Conversation

RussKie
Copy link
Member

@RussKie RussKie commented Apr 18, 2021

Resolves #9047

  • If an unhandled exception is throw on the main thread, show the error
    within the app process. The error may or may not be fatal.
    This code path is for the most part unchanged from the existing
    behaviour.

  • If an unhandled exception is thrown from a background thread it
    will crash the app. Serialize and encode the error information as base64
    string, and launch the bug reporter app before terminating the app.

  • The bug report exception handling is updated to use SerializableException
    type to accommodate both -in-app and out-of-proc exception reporting.

@ghost ghost assigned RussKie Apr 18, 2021
@RussKie RussKie added this to the 3.5 milestone Apr 18, 2021
@RussKie RussKie linked an issue Apr 18, 2021 that may be closed by this pull request
- If an unhandled exception is throw on the main thread, show the error
within the app process. The error may or may not be fatal.
This code path is for the most part unchanged from the existing
behaviour.

- If an unhandled exception is thrown from a background thread it
will crash the app. Serialize and encode the error information as base64
string, and launch the bug reporter app before terminating the app.

- The bug report exception handling is updated to use SerializableException
type to accommodate both -in-app and out-of-proc exception reporting.
@RussKie RussKie merged commit 4e923d5 into gitextensions:release/3.5 Apr 19, 2021
@RussKie RussKie deleted the bugreport_app branch April 19, 2021 03:01
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.

BugReportForm not shown for exceptions in background threads
1 participant