tmp change approach

Consensys · Nov 26, 2024 · 0cfe5bb · 0cfe5bb
1 parent 36ef653
commit 0cfe5bb
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/...erenceTest/java/tech/pegasys/teku/reference/phase0/forkchoice/StubBlobSidecarManager.java b/...erenceTest/java/tech/pegasys/teku/reference/phase0/forkchoice/StubBlobSidecarManager.java
@@ -95,11 +95,7 @@ public SafeFuture<BlobSidecarsAndValidationResult> getAvailabilityCheckResult()
         if (blobsAndProofs == null) {
           return SafeFuture.completedFuture(BlobSidecarsAndValidationResult.NOT_REQUIRED);
         }
-        try {
-          return SafeFuture.completedFuture(validateImmediately(block, blobsAndProofs));
-        } catch (RuntimeException e) {
-          return SafeFuture.completedFuture(BlobSidecarsAndValidationResult.notAvailable(e));
-        }
+        return SafeFuture.completedFuture(validateImmediately(block, blobsAndProofs));
       }
 
       private BlobSidecarsAndValidationResult validateImmediately(
@@ -111,8 +107,12 @@ private BlobSidecarsAndValidationResult validateImmediately(
                 .map(SszKZGCommitment::getKZGCommitment)
                 .toList();
         final List<Bytes> blobs = blobsAndProofs.blobs.stream().map(Blob::getBytes).toList();
-        if (!kzg.verifyBlobKzgProofBatch(blobs, kzgCommitments, blobsAndProofs.proofs)) {
-          throw new RuntimeException("Invalid KZG proof");
+        try {
+          if (!kzg.verifyBlobKzgProofBatch(blobs, kzgCommitments, blobsAndProofs.proofs)) {
+            return BlobSidecarsAndValidationResult.invalidResult(Collections.emptyList());
+          }
+        } catch (final Exception ex) {
+          return BlobSidecarsAndValidationResult.invalidResult(Collections.emptyList(), ex);
         }
         return BlobSidecarsAndValidationResult.validResult(Collections.emptyList());
       }