Skip to content

Commit

Permalink
fix expected withdrawal mismatch logging (#4798)
Browse files Browse the repository at this point in the history
  • Loading branch information
tersec authored Apr 8, 2023
1 parent ed148ab commit 7857c47
Showing 1 changed file with 12 additions and 2 deletions.
14 changes: 12 additions & 2 deletions beacon_chain/eth1/eth1_monitor.nim
Original file line number Diff line number Diff line change
Expand Up @@ -943,9 +943,19 @@ proc getPayload*(m: ELManager,
continue

if engineApiWithdrawals != req.read.executionPayload.withdrawals.maybeDeref:
# otherwise it formats as "@[(index: ..., validatorIndex: ...,
# address: ..., amount: ...), (index: ..., validatorIndex: ...,
# address: ..., amount: ...)]"
warn "Execution client did not return correct withdrawals",
withdrawals_from_cl = engineApiWithdrawals,
withdrawals_from_el = req.read.executionPayload.withdrawals
withdrawals_from_cl_len = engineApiWithdrawals.len,
withdrawals_from_el_len =
req.read.executionPayload.withdrawals.maybeDeref.len,
withdrawals_from_cl =
mapIt(engineApiWithdrawals, it.asConsensusWithdrawal),
withdrawals_from_el =
mapIt(
req.read.executionPayload.withdrawals.maybeDeref,
it.asConsensusWithdrawal)

if req.read.executionPayload.extraData.len > MAX_EXTRA_DATA_BYTES:
warn "Execution client provided a block with invalid extraData (size exceeds limit)",
Expand Down

0 comments on commit 7857c47

Please sign in to comment.