improved sidecars logging (berachain#126)

informalsystems · Jan 14, 2025 · ff4f077 · ff4f077
1 parent 002a1ff
commit ff4f077
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/beacon/blockchain/errors.go b/beacon/blockchain/errors.go
@@ -30,5 +30,7 @@ var (
 	// ErrDataNotAvailable indicates that the required data is not available.
 	ErrDataNotAvailable = errors.New("data not available")
 
+	ErrSidecarCommittmentMismatch = errors.New("sidecars commitments mismatch")
+
 	ErrSidecarSignatureMismatch = errors.New("sidecar signature mismatch")
 )
diff --git a/beacon/blockchain/process_proposal.go b/beacon/blockchain/process_proposal.go
@@ -83,9 +83,11 @@ func (s *Service) ProcessProposal(
 	// Make sure we have the right number of BlobSidecars
 	numCommitments := len(blk.GetBody().GetBlobKzgCommitments())
 	if numCommitments != len(sidecars) {
-		err = fmt.Errorf("expected %d sidecars, got %d",
+		err = fmt.Errorf("expected %d sidecars, got %d: %w",
 			numCommitments, len(sidecars),
+			ErrSidecarCommittmentMismatch,
 		)
+		s.logger.Warn(err.Error())
 		return err
 	}