skip unncessarily verifying erc1271 signature + remove stray dbg

Author: d4mr

aa-core/src/signer.rs

@@ -248,7 +248,7 @@ impl<C: Chain + Clone> SmartAccountSigner<C> {
     }
 
     /// Verify ERC-1271 signature
-    async fn verify_erc1271(&self, hash: B256, signature: &str) -> Result<bool, EngineError> {
+    pub async fn verify_erc1271(&self, hash: B256, signature: &str) -> Result<bool, EngineError> {
         let signature_bytes = hex::decode(signature.strip_prefix("0x").unwrap_or(signature))
             .map_err(|_| EngineError::ValidationError {
                 message: "Invalid signature hex".to_string(),
@@ -263,15 +263,10 @@ impl<C: Chain + Clone> SmartAccountSigner<C> {
             .await
         {
             Ok(response) => {
-                dbg!(response);
                 let expected_magic = ERC1271Contract::isValidSignatureCall::SELECTOR;
                 Ok(response.as_slice() == expected_magic)
             }
-            Err(e) => {
-                let data = e.as_revert_data().unwrap();
-                dbg!(decode_revert_reason(data.as_ref()));
-                Ok(false)
-            }
+            Err(e) => Ok(false),
         }
     }
 
@@ -333,16 +328,16 @@ impl<C: Chain + Clone> SmartAccountSigner<C> {
 
         if is_deployed {
             // Verify ERC-1271 signature for deployed accounts
-            let message_hash = self.hash_message(message, format);
-            let is_valid = self.verify_erc1271(message_hash, &signature).await?;
-
-            if is_valid {
-                Ok(signature)
-            } else {
-                Err(EngineError::ValidationError {
-                    message: "ERC-1271 signature validation failed".to_string(),
-                })
-            }
+            // let message_hash = self.hash_message(message, format);
+            // let is_valid = self.verify_erc1271(message_hash, &signature).await?;
+
+            // if is_valid {
+            Ok(signature)
+            // } else {
+            //     Err(EngineError::ValidationError {
+            //         message: "ERC-1271 signature validation failed".to_string(),
+            //     })
+            // }
         } else {
             // Create ERC-6492 signature for undeployed accounts
             self.create_erc6492_signature(&signature).await
@@ -359,21 +354,21 @@ impl<C: Chain + Clone> SmartAccountSigner<C> {
 
         if is_deployed {
             // Verify ERC-1271 signature for deployed accounts
-            let typed_data_hash =
-                typed_data
-                    .eip712_signing_hash()
-                    .map_err(|_e| EngineError::ValidationError {
-                        message: "Failed to compute typed data hash".to_string(),
-                    })?;
-            let is_valid = self.verify_erc1271(typed_data_hash, &signature).await?;
-
-            if is_valid {
-                Ok(signature)
-            } else {
-                Err(EngineError::ValidationError {
-                    message: "ERC-1271 signature validation failed".to_string(),
-                })
-            }
+            // let typed_data_hash =
+            //     typed_data
+            //         .eip712_signing_hash()
+            //         .map_err(|_e| EngineError::ValidationError {
+            //             message: "Failed to compute typed data hash".to_string(),
+            //         })?;
+            // let is_valid = self.verify_erc1271(typed_data_hash, &signature).await?;
+
+            // if is_valid {
+            Ok(signature)
+            // } else {
+            //     Err(EngineError::ValidationError {
+            //         message: "ERC-1271 signature validation failed".to_string(),
+            //     })
+            // }
         } else {
             // Create ERC-6492 signature for undeployed accounts
             self.create_erc6492_signature(&signature).await