Merge pull request #320 from 0xsequence/Feature/log-wrapper

added custom log class SequenceLog

Author: andygruening

Packages/Sequence-Unity/Sequence/SequenceSDK/Authentication/OpenIdAuthenticator.cs

@@ -282,7 +282,7 @@ public void PlatformSpecificSetup()
             catch (Exception ex)
             {
                 string message = $"Failed to register URL scheme '{_urlScheme}': {ex.Message}" + "\nSocial sign in is not currently supported on IL2CPP";
-                Debug.LogWarning(message);
+                SequenceLog.Warning(message);
                 throw new Exception(message);
             }
 #endif

Packages/Sequence-Unity/Sequence/SequenceSDK/Authentication/SignInWithApple/AppleAuth/AppleAuth.asmdef

@@ -1,3 +1,4 @@
 {
-	"name": "AppleAuth"
+	"name": "AppleAuth",
+	"references":[ "GUID:b4f9c0f8f363f439b9e337f79050f189" ]
 }

Packages/Sequence-Unity/Sequence/SequenceSDK/Authentication/SignInWithApple/AppleAuth/AppleAuthManager.cs

@@ -5,6 +5,7 @@
 using AppleAuth.Enums;
 using AppleAuth.Interfaces;
 using System;
+using Sequence.Utils;
 
 namespace AppleAuth
 {
@@ -16,7 +17,7 @@ static AppleAuthManager()
 #if APPLE_AUTH_MANAGER_NATIVE_IMPLEMENTATION_AVAILABLE
             PInvoke.AppleAuth_LogMessage(versionMessage);
 #else
-            UnityEngine.Debug.Log(versionMessage);
+            SequenceLog.Info(versionMessage);
 #endif
         }
 

Packages/Sequence-Unity/Sequence/SequenceSDK/Authentication/WebAuthSuccessPoller.cs

@@ -73,18 +73,18 @@ private async Task<bool> SendGetSuccessRequest()
             }
             catch (HttpRequestException e)
             {
-                Debug.LogError("HTTP Request failed: " + e.Message);
+                SequenceLog.Error("HTTP Request failed: " + e.Message);
             }
             catch (FormatException e)
             {
-                Debug.LogError("Invalid URL format: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
+                SequenceLog.Error("Invalid URL format: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
             }
             catch (FileLoadException e)
             {
-                Debug.LogError("File load exception: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
+                SequenceLog.Error("File load exception: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
             }
             catch (Exception e) {
-                Debug.LogError("An unexpected error occurred: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
+                SequenceLog.Error("An unexpected error occurred: " + e.Message + "\nCurl-equivalent request: " + curlRequest);
             }
 
             return false;

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ParameterTypes/SendERC721.cs

@@ -1,5 +1,6 @@
 using System;
 using Newtonsoft.Json;
+using Sequence.Utils;
 using UnityEngine;
 using UnityEngine.Scripting;
 
@@ -25,7 +26,7 @@ public SendERC721(string tokenAddress, string to, string tokenId, bool safe = tr
             this.safe = safe;
             if (!safe && data != null)
             {
-                Debug.LogError($"Error creating {GetType().Name}: {nameof(data)} can only be set when {nameof(safe)} is true.\nUsing null data.");
+                SequenceLog.Error($"Error creating {GetType().Name}: {nameof(data)} can only be set when {nameof(safe)} is true.\nUsing null data.");
                 data = null;
             }
             this.data = data;

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ParameterTypes/TransactionsValidator.cs

@@ -1,6 +1,7 @@
 using System;
 using Sequence.ABI;
 using Sequence.Contracts;
+using Sequence.Utils;
 using UnityEngine;
 
 namespace Sequence.EmbeddedWallet
@@ -23,7 +24,7 @@ public static void Validate(Transaction[] transactions)
                         if (!ABIRegex.MatchesFunctionABI(delayedEncode.data.abi))
                         {
                             string message = $"Given {nameof(DelayedEncode)} transaction with function abi {delayedEncode.data.abi} that does not match the required regex {ABIRegex.FunctionABIRegex} - for example: \"mint(uint256,uint256)\"";
-                            Debug.LogWarning(message + "\nAttempting to recover and parse anyways");
+                            SequenceLog.Warning(message + "\nAttempting to recover and parse anyways");
                             delayedEncode.data.abi = EventParser.ParseEventDef(delayedEncode.data.abi).ToString();
                             if (!ABIRegex.MatchesFunctionABI(delayedEncode.data.abi))
                             {
@@ -43,7 +44,7 @@ public static void Validate(Transaction[] transactions)
                     if (!ABIRegex.MatchesFunctionABI(contractCall.data.abi))
                     {
                         string message = $"Given {nameof(SequenceContractCall)} transaction with function abi {contractCall.data.abi} that does not match the required regex {ABIRegex.FunctionABIRegex} - for example: \"mint(uint256,uint256)\"";
-                        Debug.LogWarning(message + "\nAttempting to recover and parse anyways");
+                        SequenceLog.Warning(message + "\nAttempting to recover and parse anyways");
                         contractCall.data.abi = EventParser.ParseEventDef(contractCall.data.abi).ToString();
                         if (!ABIRegex.MatchesFunctionABI(contractCall.data.abi))
                         {

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/DataTypes/ReturnTypes/SuccessfulTransactionReturn.cs

@@ -3,6 +3,7 @@
 using Newtonsoft.Json;
 using System.Collections.Generic;
 using System;
+using Sequence.Utils;
 using UnityEngine.Scripting;
 
 namespace Sequence.EmbeddedWallet
@@ -65,7 +66,7 @@ public override object ReadJson(JsonReader reader, Type objectType, object exist
             }
             catch (Exception ex)
             {
-                UnityEngine.Debug.LogError($"Failed to load JSON: {ex.Message}");
+                SequenceLog.Error($"Failed to load JSON: {ex.Message}");
                 throw;
             }
 
@@ -91,7 +92,7 @@ public override object ReadJson(JsonReader reader, Type objectType, object exist
             }
             catch (Exception ex)
             {
-                UnityEngine.Debug.LogError($"Error during JSON conversion: {ex.Message}");
+                SequenceLog.Error($"Error during JSON conversion: {ex.Message}");
                 throw;
             }
 

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/EOAWalletLinker.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Threading.Tasks;
+using Sequence.Utils;
 using UnityEngine;
 using UnityEngine.Scripting;
 
@@ -43,7 +44,7 @@ public async Task<LinkedWalletData[]> GetLinkedWallets()
             }
             catch (Exception e)
             {
-                Debug.LogException(e);
+                SequenceLog.Exception(e);
                 return Array.Empty<LinkedWalletData>();
             }
         }
@@ -71,7 +72,7 @@ await client.SendRequest<LinkedWalletsRequestData, LinkedWalletsResponseData>(
             }
             catch (Exception e)
             {
-                Debug.LogException(e);
+                SequenceLog.Exception(e);
                 return false;
             }
         }
@@ -98,7 +99,7 @@ await client.SendRequest<NonceRequestData, NonceResponseData>("",
             }
             catch (System.Exception e)
             {
-                Debug.LogError($"Failed to generate EOA Wallet Link: {e}");
+                SequenceLog.Error($"Failed to generate EOA Wallet Link: {e}");
                 return null;
             }
         }

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/EOAWalletToSequenceWalletAdapter.cs

@@ -8,6 +8,7 @@
 using Sequence.Provider;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
+using Sequence.Utils;
 using UnityEngine;
 namespace Sequence.EmbeddedWallet
 {
@@ -171,7 +172,7 @@ public async Task<TransactionReturn> SendTransaction(Chain network, Transaction[
                     }
                     catch (Exception e)
                     {
-                        Debug.LogError("Error while waiting for receipt: " + e.Message);
+                        SequenceLog.Error("Error while waiting for receipt: " + e.Message);
                         string receiptExString = e.Message;
                         var failedReturn = new FailedTransactionReturn(e.Message, null, null);
                         failedTransactionReturns.Add(failedReturn);

Packages/Sequence-Unity/Sequence/SequenceSDK/EmbeddedWallet/HttpClient.cs

@@ -137,7 +137,8 @@ public async Task<T2> SendRequest<T, T2>(string path, T args, [CanBeNull] Dictio
             UnityWebRequest request = newRequest.Item1;
             string curlRequest = newRequest.Item2;
             string url = newRequest.Item3;
-            Debug.Log(curlRequest);
+            
+            SequenceLog.Info($">> {curlRequest}");
 
             try
             {
@@ -174,6 +175,7 @@ public async Task<T2> SendRequest<T, T2>(string path, T args, [CanBeNull] Dictio
                     }
                     try
                     {
+                        SequenceLog.Info($"<< {responseJson} (from {url})");
                         T2 result = JsonConvert.DeserializeObject<T2>(responseJson);
                         return result;
                     }
@@ -254,7 +256,7 @@ public async Task<TimeSpan> GetTimeShift()
             }
             catch (Exception e)
             {
-                Debug.LogError("Error getting time shift: " + e.Message);
+                SequenceLog.Error("Error getting time shift: " + e.Message);
                 return TimeSpan.Zero;
             }
             finally