cleanup

Author: 0xFirekeeper

Thirdweb/Thirdweb.Extensions/ThirdwebExtensions.cs

@@ -274,7 +274,6 @@ public static async Task<T> Authenticate<T>(
 
         _ = payloadResponse.EnsureSuccessStatusCode();
         var payloadString = await payloadResponse.Content.ReadAsStringAsync();
-        Console.WriteLine(payloadString);
 
         var loginBodyRaw = JsonConvert.DeserializeObject<LoginPayload>(payloadString);
         if (loginBodyRaw.Payload == null)

Thirdweb/Thirdweb.Wallets/SmartWallet/SmartWallet.cs

@@ -426,7 +426,7 @@ private async Task<object> SignUserOp(ThirdwebTransactionInput transactionInput,
             }
             else
             {
-                var res = await this.GetPaymasterAndData(requestId, EncodeUserOperation(partialUserOp), true);
+                var res = await this.GetPaymasterAndData(requestId, EncodeUserOperation(partialUserOp), simulation);
                 partialUserOp.Paymaster = res.Paymaster;
                 partialUserOp.PaymasterData = res.PaymasterData?.HexToBytes() ?? Array.Empty<byte>();
                 partialUserOp.PreVerificationGas = new HexBigInteger(res.PreVerificationGas ?? "0").Value;
@@ -718,7 +718,7 @@ public async Task<string> PersonalSign(string message)
                 : await this._personalAccount.PersonalSign(originalMsgHash);
 
             var isValid = await this.IsValidSignature(message, sig);
-            return isValid ? sig : throw new Exception("Invalid signature.");
+            return isValid ? sig : throw new Exception("Invalid signature. Outside of the signer being incorrect, this can happen if the caller if not an approved target.");
         }
         else
         {
@@ -733,16 +733,13 @@ public async Task<string> RecoverAddressFromPersonalSign(string message, string
 
     public async Task<bool> IsValidSignature(string message, string signature)
     {
-        Console.WriteLine($"IsValidSignature: {message}, {signature}");
-
         try
         {
             var magicValue = await ThirdwebContract.Read<byte[]>(this._accountContract, "isValidSignature", Encoding.UTF8.GetBytes(message).HashPrefixedMessage(), signature.HexToBytes());
             return magicValue.BytesToHex() == new byte[] { 0x16, 0x26, 0xba, 0x7e }.BytesToHex();
         }
-        catch (Exception e)
+        catch
         {
-            Console.WriteLine("Error calling isValidSignature: " + e.Message);
             return false;
         }
     }

Thirdweb/Thirdweb.Wallets/SmartWallet/Thirdweb.AccountAbstraction/BundlerClient.cs

@@ -89,10 +89,6 @@ private static async Task<RpcResponseMessage> BundlerRequest(ThirdwebClient clie
         var requestMessage = new RpcRequestMessage(requestId, method, args = args.Where(a => a != null).ToArray());
         var requestMessageJson = JsonConvert.SerializeObject(requestMessage);
 
-#if DEBUG
-        Console.WriteLine($"Bundler Request {url}: {requestMessageJson}");
-#endif
-
         var httpContent = new StringContent(requestMessageJson, System.Text.Encoding.UTF8, "application/json");
 
         ThirdwebHttpResponseMessage httpResponse;
@@ -112,10 +108,6 @@ private static async Task<RpcResponseMessage> BundlerRequest(ThirdwebClient clie
 
         var httpResponseJson = await httpResponse.Content.ReadAsStringAsync();
 
-#if DEBUG
-        Console.WriteLine($"Bundler Response: {httpResponseJson}");
-#endif
-
         var response = JsonConvert.DeserializeObject<RpcResponseMessage>(httpResponseJson);
         return response.Error != null ? throw new Exception($"Bundler Request Failed. Error: {response.Error.Code} - {response.Error.Message} - {response.Error.Data}") : response;
     }