feat: add beacon_blocks_by_head reqresp#9331
Conversation
There was a problem hiding this comment.
Code Review
This pull request implements the BeaconBlocksByHead request-response protocol, which allows peers to request a sequence of blocks starting from a specific root and traversing backwards through parent roots. The changes include the definition of the new protocol, implementation of the onBeaconBlocksByHead handler with validation and rate limiting, and the addition of corresponding network interface methods. Unit tests have been added to verify the handler's behavior across fork boundaries and its adherence to request limits. I have no feedback to provide.
Performance Report✔️ no performance regression detected Full benchmark results
|
twoeths
left a comment
There was a problem hiding this comment.
looks good to me, I dropped some minor comments
|
some unit tests are failing.. |
Motivation
beacon_blocks_by_headReqResp ethereum/consensus-specs#5181Description
AI Assistance Disclosure