should not be unwrapping here (#1125)

xmtp · Oct 9, 2024 · c44c8ab · c44c8ab
1 parent 137062e
commit c44c8ab
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/xmtp_id/src/scw_verifier/mod.rs b/xmtp_id/src/scw_verifier/mod.rs
@@ -29,6 +29,8 @@ pub enum VerifierError {
     Abi(#[from] ethers::abi::Error),
     #[error(transparent)]
     Provider(#[from] ethers::providers::ProviderError),
+    #[error(transparent)]
+    Tonic(tonic::Status),
 }
 
 #[async_trait]

diff --git a/xmtp_id/src/scw_verifier/remote_signature_verifier.rs b/xmtp_id/src/scw_verifier/remote_signature_verifier.rs
@@ -48,7 +48,7 @@ impl SmartContractSignatureVerifier for RemoteSignatureVerifier {
                 }],
             })
             .await
-            .unwrap();
+            .map_err(VerifierError::Tonic)?;
 
         let VerifySmartContractWalletSignaturesResponse { responses } = result.into_inner();