fix: avoid attempting to flush unhandled error reports in same session

bugsnag · Jul 15, 2020 · a65addf · a65addf
1 parent e6d6890
commit a65addf
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,10 @@
 # Changelog
 
+## TBD
+
+* Avoid attempting to flush unhandled error reports in same session
+  [#902](https://github.com/bugsnag/bugsnag-android/pull/902)
+
 ## 5.0.0 (2020-04-22)
 
 __This version contains many breaking changes__. It is part of an effort to unify our notifier

diff --git a/bugsnag-android-core/src/main/java/com/bugsnag/android/DeliveryDelegate.java b/bugsnag-android-core/src/main/java/com/bugsnag/android/DeliveryDelegate.java
@@ -43,12 +43,23 @@ void deliver(@NonNull Event event) {
         }
 
         if (event.isUnhandled()) {
-            cacheEvent(event, true);
+            cacheEvent(event, isAnr(event));
         } else {
             deliverPayloadAsync(event, eventPayload);
         }
     }
 
+    private boolean isAnr(@NonNull Event event) {
+        List<Error> errors = event.getErrors();
+        String errorClass = null;
+
+        if (!errors.isEmpty()) {
+            Error error = errors.get(0);
+            errorClass = error.getErrorClass();
+        }
+        return "ANR".equals(errorClass);
+    }
+
     private void deliverPayloadAsync(@NonNull Event event, EventPayload eventPayload) {
         final EventPayload finalEventPayload = eventPayload;
         final Event finalEvent = event;