From b7d08d0a38cb1259432350e3eee888822b60fe23 Mon Sep 17 00:00:00 2001 From: Etan Kissling Date: Sun, 9 Apr 2023 16:58:20 +0200 Subject: [PATCH] do not report pre-Merge sync progress as `/opt` (#4801) Before the merge, assume `payloadStatus == NewPaylodStatus.valid` to avoid cases of sync progress being reported with `/opt` suffix. --- beacon_chain/gossip_processing/block_processor.nim | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/beacon_chain/gossip_processing/block_processor.nim b/beacon_chain/gossip_processing/block_processor.nim index 4a820586b..f719128a8 100644 --- a/beacon_chain/gossip_processing/block_processor.nim +++ b/beacon_chain/gossip_processing/block_processor.nim @@ -349,7 +349,8 @@ proc storeBlock*( payloadStatus = if maybeFinalized and (self.lastPayload + SLOTS_PER_PAYLOAD) > signedBlock.message.slot and - (signedBlock.message.slot + PAYLOAD_PRE_WALL_SLOTS) < wallSlot: + (signedBlock.message.slot + PAYLOAD_PRE_WALL_SLOTS) < wallSlot and + signedBlock.message.is_execution_block: # Skip payload validation when message source (reasonably) claims block # has been finalized - this speeds up forward sync - in the worst case # that the claim is false, we will correct every time we process a block