fix(walletconnect): fix WalletConnectRpc.GetRandomId throwing error…

… when not in main thread
CareBoo · Jun 12, 2022 · 133fbd6 · 133fbd6
1 parent 6f23d10
commit 133fbd6
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/Runtime/CareBoo.AlgoSdk.WalletConnect/Models/WalletConnectRpc.cs b/Runtime/CareBoo.AlgoSdk.WalletConnect/Models/WalletConnectRpc.cs
@@ -1,4 +1,5 @@
 using System;
+using Unity.Mathematics;
 
 namespace AlgoSdk.WalletConnect
 {
@@ -50,8 +51,14 @@ public static JsonRpcRequest SessionRequest(WalletConnectSessionRequest sessionR
         /// <summary>
         /// Gets a random, valid JsonRpcRequest id.
         /// </summary>
-        /// <returns>a <see cref="ulong"/> in the range [1, <see cref="int.MaxValue"/>]</returns>
-        public static ulong GetRandomId() => (ulong)UnityEngine.Random.Range(1, int.MaxValue);
+        /// <returns>a <see cref="ulong"/> in the range [1, <see cref="ulong.MaxValue"/>]</returns>
+        public static ulong GetRandomId()
+        {
+            ulong x = 0;
+            while (x == 0)
+                x = AlgoSdk.Crypto.Random.Bytes<ulong>();
+            return x;
+        }
 
         public static class Algorand
         {