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

Apply proposer boost to first block in equivocation #12947

Closed
wants to merge 4 commits into from
Closed

Conversation

potuz
Copy link
Contributor

@potuz potuz commented Sep 22, 2023

This PR applies ethereum/consensus-specs#3352

This is not strictly necessary in Prysm due to the way we process blocks, we would not process the second block of the equivocation until we have already passed the proposer Boost threshold. However, it's safer to explicitly do it, and in any case we need it to pass spectests.

Fixes: #12946

@potuz potuz added Ready For Review A pull request ready for code review OK to merge labels Sep 22, 2023
@potuz potuz requested a review from a team as a code owner September 22, 2023 18:40
@potuz potuz removed the OK to merge label Sep 23, 2023
@potuz
Copy link
Contributor Author

potuz commented Sep 28, 2023

closing as this is included in #12959

@potuz potuz closed this Sep 28, 2023
@terencechain terencechain deleted the 3352 branch May 13, 2024 04:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply proposer boost to first block in case of equivocation
2 participants