Merge pull request #225 from matter-labs/handle_old_proof_response

fix(proof-validation): handle optional proof status
matter-labs · Nov 28, 2024 · e4629ae · e4629ae
2 parents 6e88e20 + 78471f5
commit e4629ae
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/bin/verify-era-proof-attestation/src/main.rs b/bin/verify-era-proof-attestation/src/main.rs
@@ -159,7 +159,10 @@ async fn verify_batch_proofs(
         total_proofs_count += 1;
         let tee_type = proof.tee_type.to_uppercase();
 
-        if proof.status.eq_ignore_ascii_case("permanently_ignored") {
+        if proof
+            .status
+            .map_or(false, |s| s.eq_ignore_ascii_case("permanently_ignored"))
+        {
             trace!(
                 batch_no,
                 tee_type,

diff --git a/bin/verify-era-proof-attestation/src/proof.rs b/bin/verify-era-proof-attestation/src/proof.rs
@@ -39,8 +39,9 @@ pub async fn get_proofs(
 
         if !proofs.is_empty()
             && proofs.iter().all(|proof| {
-                !proof.status.eq_ignore_ascii_case("failed")
-                    && !proof.status.eq_ignore_ascii_case("picked_by_prover")
+                !proof.status.as_ref().map_or(false, |s| {
+                    s.eq_ignore_ascii_case("failed") | s.eq_ignore_ascii_case("picked_by_prover")
+                })
             })
         {
             return Ok(proofs);
@@ -165,7 +166,7 @@ pub struct Proof {
     #[serde_as(as = "Option<Hex>")]
     pub proof: Option<Vec<u8>>,
     pub proved_at: String,
-    pub status: String,
+    pub status: Option<String>,
     #[serde_as(as = "Option<Hex>")]
     pub attestation: Option<Vec<u8>>,
 }