Skip to content
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

Som/pectra4 engine api #239

Merged
merged 7 commits into from
Oct 23, 2024
Merged

Som/pectra4 engine api #239

merged 7 commits into from
Oct 23, 2024

Conversation

somnathb1
Copy link
Contributor

No description provided.

somnathb1 added a commit to erigontech/erigon that referenced this pull request Oct 23, 2024
Summary of changes
- Remove `Requests` interface in favour of new `FlatRequest` struct
- Add changes for new `RequestHash` calculation that `sha256` digests
the set of flat requests
- Remove `Requests` from block and body related structs and methods
- Set of requests that gets pulled at the `Finalize` stage is now
returned from there, both for execution and block-building

Ref1: ethereum/execution-apis#591
Ref2: ethereum/EIPs#8854
Ref3: ethereum/EIPs#8924


Needs interface change -
erigontech/interfaces#239

(Tasks board - #12106)
@yperbasis yperbasis merged commit c6c0be2 into master Oct 23, 2024
@yperbasis yperbasis deleted the som/pectra4_engine_api branch October 23, 2024 13:23
yperbasis pushed a commit that referenced this pull request Oct 24, 2024
* Flatten 7251 req

* Replace requests with generic requests

* fix type

* change byte to uint8

* Simplify requests bundle

* Rename requests_hash and remove reqs from body
@yperbasis yperbasis mentioned this pull request Oct 24, 2024
yperbasis added a commit that referenced this pull request Oct 24, 2024
* Flatten 7251 req

* Replace requests with generic requests

* fix type

* change byte to uint8

* Simplify requests bundle

* Rename requests_hash and remove reqs from body

Co-authored-by: Somnath <snb895@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants