Skip to content
This repository has been archived by the owner on Feb 18, 2025. It is now read-only.

Compute withdrawals root in execution payload validation #56

Merged

Conversation

emhane
Copy link
Member

@emhane emhane commented Jan 29, 2025

Defines OpExecutionPayloadValidator for replacing use of l1 ExecutionPayloadValidator in OpEngineValidator. New type doesn't compute withdrawals root from empty withdrawals list, but instead has reference to the database and retrieves storage root of l2tol1 message passing contract instead.

@emhane emhane added D-isthmus-devnet-0 Devnet: l2 withdrawals root devnet A-consensus Related to consensus engine labels Jan 29, 2025
@emhane emhane force-pushed the emhane/op-exec-payload-validator branch from 495031a to c34b18b Compare January 30, 2025 18:59
@emhane emhane merged commit 27e41aa into emhane/exec-verif-storage-root Jan 30, 2025
19 of 26 checks passed
@emhane emhane deleted the emhane/op-exec-payload-validator branch January 30, 2025 19:19
@emhane emhane added the upstream A pr solving this issue has been opened in upstream label Feb 3, 2025
@emhane emhane restored the emhane/op-exec-payload-validator branch February 4, 2025 13:27
@emhane emhane deleted the emhane/op-exec-payload-validator branch February 4, 2025 16:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-consensus Related to consensus engine D-isthmus-devnet-0 Devnet: l2 withdrawals root devnet upstream A pr solving this issue has been opened in upstream
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant