status-im
/
nimbus-eth1
mirror of https://github.com/status-im/nimbus-eth1.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

Disable canonical verification for blockheaders post-merge (#1377)

This commit is contained in:
Kim De Mey 2022-12-15 14:13:26 +01:00 committed by GitHub
parent e07898f949
commit 706effea2a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
fluffy/network/history/accumulator.nim
View File

@ -170,7 +170,13 @@ func verifyHeader*(
of BlockHeaderProofType.accumulatorProof: of BlockHeaderProofType.accumulatorProof:
a.verifyAccumulatorProof(header, proof.accumulatorProof) a.verifyAccumulatorProof(header, proof.accumulatorProof)
of BlockHeaderProofType.none: of BlockHeaderProofType.none:
err("cannot verify header without proof") # TODO:
# Currently there is no proof solution for verifying headers post-merge.
# Skipping canonical verification will allow for nodes to push block data
# that is not part of the canonical chain.
# For now we accept this flaw as the focus lies on testing data availability
# up to the head of the chain.
ok()
func buildProof*( func buildProof*(
header: BlockHeader, header: BlockHeader,