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

Prepare fluffy for Deneb/Cancun fork #1707

Closed
kdeme opened this issue Aug 23, 2023 · 1 comment
Closed

Prepare fluffy for Deneb/Cancun fork #1707

kdeme opened this issue Aug 23, 2023 · 1 comment
Labels

Comments

@kdeme
Copy link
Contributor

kdeme commented Aug 23, 2023

Adjust to any changes required here

@kdeme kdeme added the Fluffy label Aug 23, 2023
@kdeme kdeme changed the title Prepare fluffy for Deneb fork Prepare fluffy for Deneb/Cancun fork Aug 23, 2023
@kdeme
Copy link
Contributor Author

kdeme commented Feb 9, 2024

Looking at this, it seems nothing more than this PR is required: #2016
As long as we can properly decode the updated RLP BlockHeader, it should be good. (And this is supposed to be even compatible to addition of fields).

Looks like nothing should change to the SSZ BlockBody (nor Receipts) container for this fork. Blob data is ephemeral and not part of the BlockBody, so we should not store this in history network (It could go in a separate Portal network if wanted).

There is a new transaction type, but we already have transactions as RLP bytestrings in the SSZ container.
And even if we would need to decode them, we do support the new type in nim-eth.

@kdeme kdeme closed this as completed Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant