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

Task: Update EigenPod Docs #417

Open
2 tasks
Sidu28 opened this issue Feb 1, 2024 · 0 comments
Open
2 tasks

Task: Update EigenPod Docs #417

Sidu28 opened this issue Feb 1, 2024 · 0 comments
Assignees

Comments

@Sidu28
Copy link
Contributor

Sidu28 commented Feb 1, 2024

Description

Update EigenPod docs to relfect the new capella->deneb switch in the EigenPodManager that turns on the deneb-specific proof path

Some context on the change:
As per deneb upgrade, an additional 2 fields are added to the ExecutionPayloadHeader container which means that it merkelizes differently (theres an extra layer to the tree), specifically the height of the merkleized ExecutionPayloadHeader is now 5 instead of 4. These two fields however are not connected to our proofs and do not change them. See the deneb updates here.

This PR adds a switch that allows us to set a deneb fork timestamp twice - once from 0 -> to type(uint64).max and then a second time type(uint64).max -> actual_deneb_fork_timestamp. That way, capella withdrawal proofs work along with deneb withdrawal proofs

Action Items

  • Add doc edits
  • Get review

Blocking Issues

n/a

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

No branches or pull requests

2 participants