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

Doppelgänger Protection Feature #3689

Closed
5 tasks done
philknows opened this issue Feb 1, 2022 · 0 comments · Fixed by #3883
Closed
5 tasks done

Doppelgänger Protection Feature #3689

philknows opened this issue Feb 1, 2022 · 0 comments · Fixed by #3883
Assignees
Labels
prio-medium Resolve this some time soon (tm). scope-ux Issues for CLI UX or general consumer UX.

Comments

@philknows
Copy link
Member

philknows commented Feb 1, 2022

Is your feature request related to a problem? Please describe.
Feature Request as part of #3687 RocketPool Integration and safety for validators.

Describe the solution you'd like
Reference implementation from Lighthouse: https://lighthouse-book.sigmaprime.io/validator-doppelganger.html

Miss 2 attestations upon startup and listen for attestations coming from the loaded keys, disable validation duties if found because it would be a slashable offense to double attest.

  • Expose liveness API inspired by Lighthouse's implementation
  • Add the doppelganger service which polls BN's liveness API and keeps info about validators
  • Use the doppelganger in validator and check its doppelganger state before performing any slash-able action
  • Reasonable test coverage
  • Add metrics
@philknows philknows added the prio-high Resolve issues as soon as possible. label Feb 11, 2022
@dapplion dapplion added scope-ux Issues for CLI UX or general consumer UX. prio-medium Resolve this some time soon (tm). and removed prio-high Resolve issues as soon as possible. labels May 10, 2022
@dapplion dapplion added this to the Audit 2022Q2-batch2 milestone Jun 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio-medium Resolve this some time soon (tm). scope-ux Issues for CLI UX or general consumer UX.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants