-
Notifications
You must be signed in to change notification settings - Fork 1.1k
eip7928: introduce specs for block access lists #4526
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eip7928: introduce specs for block access lists #4526
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this PR should include tests
@nerolation it's pretty clear that this PR was done by an LLM. Mind cleaning up after it? Eg this makes no sense and the linter check is still failing:
|
tests/core/pyspec/eth2spec/test/eip7928/test_block_access_list.py
Outdated
Show resolved
Hide resolved
Thanks @jtraglia! |
Co-authored-by: Justin Traglia <[email protected]>
Add EIP-7928: Block-Level Access Lists
This PR implements EIP-7928, which introduces block-level access lists. The implementation extends the
ExecutionPayload
andExecutionPayloadHeader
structures to include a newblock_access_list
field that contains RLP-encoded access list data. This allows execution clients to provide and validate block-level access lists through the existing Engine API interface without requiring new methods.make lint
to check formatting (completed with some unrelated prysm warnings)make test
to check tests (running)Related to Ethereum EIP-7928: https://eips.ethereum.org/EIPS/eip-7928
Summary of Changes
ExecutionPayload
container withblock_access_list: BlockAccessList
fieldExecutionPayloadHeader
container withblock_access_list_root: Root
fieldprocess_execution_payload
to compute and store the access list rootfork.md
for upgrading to EIP-7928