X Tutup
The Wayback Machine - https://web.archive.org/web/20201204180828/https://github.com/prysmaticlabs/prysm/pull/8024
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

[Feature] - Slashing Interchange Support #8024

Draft
wants to merge 9 commits into
base: develop
from

Conversation

@rauljordan
Copy link
Contributor

@rauljordan rauljordan commented Dec 2, 2020

What type of PR is this?

Feature

What does this PR do? Why is it needed?

This PR attempts to merge a feature branch feature/slashing-interchange into develop, tracking Prysm's support and integration for EIP-3076: the slashing protection interchange standard for eth2 clients.

Which issues(s) does this PR fix?

Fixes #7813

rauljordan added 3 commits Dec 2, 2020
…tection (#8020)

* amend to use bools

* ineff assign

* comment
terencechain and others added 6 commits Dec 2, 2020
* Replace highest with lowerest

* Update validator/db/kv/attestation_history_v2.go

Co-authored-by: Raul Jordan <raul@prysmaticlabs.com>

* Update validator/db/kv/attestation_history_v2.go

Co-authored-by: Raul Jordan <raul@prysmaticlabs.com>

* Invert equality for saveLowestSourceTargetToDB

* Add eip checks to ensure epochs cant be lower than db ones

* Should be less than equal to

* Check if epoch exists in DB getters

* Revert run time checks

Co-authored-by: Raul Jordan <raul@prysmaticlabs.com>
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
* added in att history checks

* logic for export

* export return nil

* test for export atts

* round trip passes first try!

* rem println

* fix up tests

* pass test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.
X Tutup