From a6ea98a0967a4ccfceac259173ecc22bb1061dd4 Mon Sep 17 00:00:00 2001 From: Harald Hoyer Date: Fri, 28 Feb 2025 16:50:22 +0100 Subject: [PATCH] fix(verify-era-proof-attestation): handle missing RecoveryId signatures - add `RecoveryId::Two` and `RecoveryId::Three` Signed-off-by: Harald Hoyer --- bin/verify-era-proof-attestation/src/verification.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/bin/verify-era-proof-attestation/src/verification.rs b/bin/verify-era-proof-attestation/src/verification.rs index 6b439f0..5603fb2 100644 --- a/bin/verify-era-proof-attestation/src/verification.rs +++ b/bin/verify-era-proof-attestation/src/verification.rs @@ -55,7 +55,12 @@ impl TeeProof { if self.signature.len() == 64 { info!("Signature is missing RecoveryId!"); // Fallback for missing RecoveryId - for rec_id in [RecoveryId::Zero, RecoveryId::One] { + for rec_id in [ + RecoveryId::Zero, + RecoveryId::One, + RecoveryId::Two, + RecoveryId::Three, + ] { let Ok(sig) = RecoverableSignature::from_compact(&sig_vec, rec_id) else { continue; };