reporting: privent noisy reports

deckerst · Jan 26, 2024 · 55e8427 · 55e8427
1 parent 2e13879
commit 55e8427
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 14 deletions.
diff --git a/lib/image_providers/thumbnail_provider.dart b/lib/image_providers/thumbnail_provider.dart
@@ -1,6 +1,7 @@
 import 'dart:ui' as ui;
 
 import 'package:aves/services/common/services.dart';
+import 'package:aves_report/aves_report.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/widgets.dart';
@@ -46,14 +47,14 @@ class ThumbnailProvider extends ImageProvider<ThumbnailProviderKey> {
         taskKey: key,
       );
       if (bytes.isEmpty) {
-        throw StateError('$uri ($mimeType) loading failed');
+        throw UnreportedStateError('$uri ($mimeType) loading failed');
       }
       final buffer = await ui.ImmutableBuffer.fromUint8List(bytes);
       return await decode(buffer);
     } catch (error) {
       // loading may fail if the provided MIME type is incorrect (e.g. the Media Store may report a JPEG as a TIFF)
       debugPrint('$runtimeType _loadAsync failed with mimeType=$mimeType, uri=$uri, error=$error');
-      throw StateError('$mimeType decoding failed (page $pageId)');
+      throw UnreportedStateError('$mimeType decoding failed (page $pageId)');
     }
   }
 

diff --git a/lib/image_providers/uri_image_provider.dart b/lib/image_providers/uri_image_provider.dart
@@ -2,6 +2,7 @@ import 'dart:async';
 import 'dart:ui' as ui;
 
 import 'package:aves/services/common/services.dart';
+import 'package:aves_report/aves_report.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/widgets.dart';
@@ -64,14 +65,14 @@ class UriImage extends ImageProvider<UriImage> with EquatableMixin {
         },
       );
       if (bytes.isEmpty) {
-        throw StateError('$uri ($mimeType) loading failed');
+        throw UnreportedStateError('$uri ($mimeType) loading failed');
       }
       final buffer = await ui.ImmutableBuffer.fromUint8List(bytes);
       return await decode(buffer);
     } catch (error) {
       // loading may fail if the provided MIME type is incorrect (e.g. the Media Store may report a JPEG as a TIFF)
       debugPrint('$runtimeType _loadAsync failed with mimeType=$mimeType, uri=$uri, error=$error');
-      throw StateError('$mimeType decoding failed (page $pageId)');
+      throw UnreportedStateError('$mimeType decoding failed (page $pageId)');
     } finally {
       unawaited(chunkEvents.close());
     }

diff --git a/lib/services/geocoding_service.dart b/lib/services/geocoding_service.dart
@@ -1,7 +1,6 @@
 import 'dart:async';
 import 'dart:ui';
 
-import 'package:aves/services/common/services.dart';
 import 'package:equatable/equatable.dart';
 import 'package:flutter/foundation.dart';
 import 'package:flutter/services.dart';
@@ -22,14 +21,8 @@ class GeocodingService {
         'maxResults': 2,
       });
       return (result as List).cast<Map>().map(Address.fromMap).toList();
-    } on PlatformException catch (e, stack) {
-      if (!{
-        'getAddress-empty',
-        'getAddress-network',
-        'getAddress-unavailable',
-      }.contains(e.code)) {
-        await reportService.recordError(e, stack);
-      }
+    } on PlatformException catch (_) {
+      // do not report
     }
     return [];
   }

diff --git a/plugins/aves_report/lib/aves_report.dart b/plugins/aves_report/lib/aves_report.dart
@@ -35,3 +35,7 @@ abstract class ReportService {
         .join('\n'));
   }
 }
+
+class UnreportedStateError extends StateError {
+  UnreportedStateError(super.message);
+}
diff --git a/plugins/aves_report_crashlytics/lib/aves_report_platform.dart b/plugins/aves_report_crashlytics/lib/aves_report_platform.dart
@@ -71,7 +71,9 @@ class PlatformReportService extends ReportService {
     if (exception is PlatformException && stack != null) {
       stack = ReportService.buildReportStack(stack, level: 2);
     }
-    return _instance?.recordError(exception, stack);
+    if (exception! is UnreportedStateError) {
+      return _instance?.recordError(exception, stack);
+    }
   }
 
   @override