Skip to content

Commit

Permalink
Fix fetch blobs in all-null case (#6940)
Browse files Browse the repository at this point in the history
Fix another issue with fetch-blobs, similar to:

- #6911


  Check if the list of blobs returned is all `None`, and if so, do not proceed any further.

This prevents an ugly error like:

> Feb 03 17:32:12.384 ERRO Error fetching or processing blobs from EL, block_root: 0x7326fe2dc1cb9036c9de7a07a662c86a339085597849016eadf061b70b7815ba, error: BlobProcessingError(AvailabilityCheck(Unexpected)), module
: network::network_beacon_processor:1011
  • Loading branch information
michaelsproul authored Feb 7, 2025
1 parent d6596db commit cb117f8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion beacon_node/beacon_chain/src/fetch_blobs.rs
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ pub async fn fetch_and_process_engine_blobs<T: BeaconChainTypes>(
.await
.map_err(FetchEngineBlobError::RequestFailed)?;

if response.is_empty() {
if response.is_empty() || response.iter().all(|opt| opt.is_none()) {
debug!(
log,
"No blobs fetched from the EL";
Expand Down

0 comments on commit cb117f8

Please sign in to comment.