Merge pull request #87 from bugsnag/release-v1.2.1

Release v1.2.1

CHANGELOG.md

@@ -1,6 +1,12 @@
 Changelog
 =========
 
+## 1.2.1 (2024-09-18)
+
+### Bug fixes
+
+* Fixed an exception thrown when no source of cryptographically secure random numbers is available [86](https://github.com/bugsnag/bugsnag-flutter-performance/pull/86)
+
 ## 1.2.0 (2024-09-13)
 
 ### Enhancements

VERSION

@@ -1 +1 @@
-1.2.0
+1.2.1

packages/bugsnag_flutter_performance/lib/src/extensions/resource_attributes.dart

@@ -196,5 +196,5 @@ class ResourceAttributesProviderImpl implements ResourceAttributesProvider {
     return "Unknown";
   }
 
-  static String get _getSDKVersion => '1.2.0';
+  static String get _getSDKVersion => '1.2.1';
 }

packages/bugsnag_flutter_performance/lib/src/util/random.dart

@@ -1,19 +1,56 @@
 import 'dart:math';
 
 BigInt randomSpanId() {
-  return randomValue(8);
+  return randomValue(8, false);
 }
 
 BigInt randomTraceId() {
-  return randomValue(16);
+  return randomValue(16, true);
 }
 
-BigInt randomValue(int length) {
-  final random = Random.secure();
+BigInt randomValue(int length, bool secure) {
+  final random = _Random(secure);
   BigInt result = BigInt.from(random.nextInt(256) & 0xff);
   for (var i = 1; i < length; i++) {
     int byte = random.nextInt(256);
     result = (result << 8) | BigInt.from(byte & 0xff);
   }
   return result;
 }
+
+class _Random {
+  static final _sharedRandom = _createSharedRandom();
+  late final Random? _secureRandom;
+  final bool secure;
+
+  _Random(this.secure) {
+    try {
+      if (secure) {
+        _secureRandom = Random.secure();
+      }
+    } catch (_) {
+      // If the platform doesn't provide a secure random source then we fall back on a less secure version
+    }
+  }
+
+  int nextInt(int max) {
+    try {
+      if (_secureRandom != null) {
+        return _secureRandom!.nextInt(max);
+      }
+    } catch (_) {
+      // When the system runs out of entropy, we fall back on a less secure random number source
+      // Although the documentation doesn't mention it, nextInt can throw
+    }
+    return _sharedRandom.nextInt(max);
+  }
+
+  static Random _createSharedRandom() {
+    try {
+      final secureRandom = Random.secure();
+      return Random(secureRandom.nextInt(1 << 32));
+    } catch (_) {
+      return Random();
+    }
+  }
+}

packages/bugsnag_flutter_performance/pubspec.yaml

@@ -1,6 +1,6 @@
 name: bugsnag_flutter_performance
 description: BugSnag performance monitoring tool for Flutter apps
-version: 1.2.0
+version: 1.2.1
 homepage: https://www.bugsnag.com/
 documentation: https://docs.bugsnag.com/performance/flutter/
 repository: https://github.com/bugsnag/bugsnag-flutter-performance