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

crashpad/android: crash loop detection, guid override #63

Merged
merged 2 commits into from
May 30, 2024

Conversation

gm4sl
Copy link
Collaborator

@gm4sl gm4sl commented May 29, 2024

While updating backtrace-android to use StartJavaHandlerAtCrash it was noticed that arguments for crash loop detection and guid override were not being passed through to the crashed handler.

This commit adds these arguments in StartJavaHandlerAtCrash to support these features when using this Android-specific mechanism and are of the form:

  • for crash loop detection: --annotation=run-uuid=
  • for guid override: --annotation=_backtrace_internal_guid_override=

Internal ref BT-2929

@gm4sl gm4sl requested a review from konraddysput May 29, 2024 17:16
gm4sl added 2 commits May 30, 2024 10:36
Factor the appending of additional arguments that are passed to a handler
for the crash loop detection and guid override features. These will also
be passed to the Java handler used by Android in a later commit.

Internal ref BT-2929
While updating backtrace-android to use StartJavaHandlerAtCrash it was
noticed that arguments for crash loop detection and guid override were not
being passed through to the crashpad handler.

This commit adds these arguments in `StartJavaHandlerAtCrash` and have
the form:

 * for crash loop detection: `--annotation=run-uuid=`
 * for guid override: `--annotation=_backtrace_internal_guid_override=`

Internal ref BT-2929
@gm4sl gm4sl force-pushed the gm4sl/BT-2929_android_handler_args branch from cd4da72 to 76e3b6f Compare May 30, 2024 14:36
@gm4sl gm4sl merged commit 7c798c0 into backtrace May 30, 2024
15 checks passed
@gm4sl gm4sl deleted the gm4sl/BT-2929_android_handler_args branch May 30, 2024 14:44
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