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

fix default value when voting on eth1data #1562

Merged
merged 1 commit into from
Jan 9, 2020
Merged

Conversation

djrtwo
Copy link
Contributor

@djrtwo djrtwo commented Jan 9, 2020

Thank you @cemozerr for finding and reporting this.

If the eth1 chain is live and well connected, the validator votes on the 0th block in the acceptable voting period when no other valid votes already exist.

If the eth1 chain is not live, the validator defaults to voting on the previous eth1data which is a no-op

@djrtwo djrtwo force-pushed the eth1-default-bug branch 3 times, most recently from 681b755 to 3a67380 Compare January 9, 2020 21:51
Copy link
Contributor

@protolambda protolambda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Nice to be more explicit about sort order and a safe default vote.

@protolambda protolambda merged commit a41447a into dev Jan 9, 2020
@protolambda protolambda deleted the eth1-default-bug branch January 9, 2020 21:57
@djrtwo djrtwo changed the title [WIP] fix default value when voting on eth1data fix default value when voting on eth1data Jan 9, 2020
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