firebase / firebase/firebase-android-sdk

Allow to include all threads for non-fatal exceptions

Open
#3,284 2 comments 0 reactions 0 assignees View on GitHub
api: crashlytics type: feature request
Dominant language
Java
Stars
2.6k
Forks
710
Avg merge
2d 23h
Merged PRs (30d)
34

Description

## What feature would you like to see?

I suggest to add an extra argument `includeAllThreads` to `recordException`:
```
public class FirebaseCrashlytics
{
...
public void recordException(Throwable throwable, boolean includeAllThreads)
...
}
```

## How would you use it?

We are investigating ANRs using ANR-WatchDog library.
The default behaviour of the `ANRWatchDog` class is to throw unhandled exception which crashes the app after `main` thread is blocked for the specified amount of time. When it happens Crashlytics collects all threads stack-traces and produces a Fatal crash report.

This behaviour naturally boosts crash-rate of the app what is not good. But when I handle `ANRWatchDog`'s exception to record it using `FirebaseCrashlytics.recordException` it does not include other threads stack-traces by design. So basically such non-fatal crash report is useless in ANR investigations.

I would like to force Crashlytics to collect all threads stack-traces and produce full crash report for a non-fatal exception.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.