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

Prepare validator DB for attester protection implementation #4584

Merged
merged 10 commits into from Jan 19, 2020

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented Jan 18, 2020

This PR modifies the validator DB so the attester protection can be fully implemented. This only includes the DB functions with no implementation added yet.

Functions added to validator DB:

  • AttestationHistory
  • SaveAttestationHistory
  • DeleteAttestationHistory

Implementation will be completed in another PR.

Design doc can be found here: https://hackmd.io/EWjipgMxQ-ScOXczfLR2gQ?view

@0xKiwi 0xKiwi added the Ready For Review A pull request ready for code review label Jan 18, 2020
@codecov
Copy link

codecov bot commented Jan 18, 2020

Codecov Report

Merging #4584 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4584   +/-   ##
=======================================
  Coverage   17.26%   17.26%           
=======================================
  Files          94       94           
  Lines        5820     5820           
=======================================
  Hits         1005     1005           
  Misses       4659     4659           
  Partials      156      156

@prylabs-bulldozer prylabs-bulldozer bot merged commit a4db560 into prysmaticlabs:master Jan 19, 2020
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 24, 2020
…clabs#4584)

* Add flag for attester protection
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Remove flags
* Add attestation history DB functions to validator client
* Fix comments
* Update interface to new funcs
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Fix test
* Merge branch 'master' into protecc-attester-db
cryptomental pushed a commit to cryptomental/prysm that referenced this pull request Feb 28, 2020
…clabs#4584)

* Add flag for attester protection
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Remove flags
* Add attestation history DB functions to validator client
* Fix comments
* Update interface to new funcs
* Merge branch 'master' of https://github.com/prysmaticlabs/Prysm into protecc-attester-db
* Fix test
* Merge branch 'master' into protecc-attester-db
@0xKiwi 0xKiwi deleted the protecc-attester-db branch March 15, 2020 15:04
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.

None yet

2 participants