Skip to content

Commit

Permalink
consider blocks with deposits in eth1 data vote (#3944)
Browse files Browse the repository at this point in the history
When importing blocks with deposits from the EL, the timestamp is never
initialized for them. Therefore, only blocks without deposits (for which
the timestamp is obtained) are considered for `is_candidate_block`.
This is fixed by also importing timestamps for blocks with deposits.
  • Loading branch information
etan-status authored Aug 9, 2022
1 parent 4ef621f commit ede83b1
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions beacon_chain/eth1/eth1_monitor.nim
Original file line number Diff line number Diff line change
Expand Up @@ -1174,6 +1174,7 @@ proc syncBlockRange(m: Eth1Monitor,

for i in 0 ..< blocksWithDeposits.len:
let blk = blocksWithDeposits[i]
awaitWithRetries m.dataProvider.fetchTimestamp(blk)

if blk.number > fullSyncFromBlock:
let lastBlock = m.depositsChain.blocks.peekLast
Expand Down

0 comments on commit ede83b1

Please sign in to comment.