-
Notifications
You must be signed in to change notification settings - Fork 975
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
PoS - verify validator keys ownership with signatures #106
Comments
another way would be to have some "challenge" in the ledger's state that Other PKs we might want to verify are the protocol key and eth bridge cold and hot key. For genesis validators, we might need to do something else as those are created before there's any ledger or chain ID. We can e.g. have some pseudo random hash that we ask validators to sign over with those keys. |
@brentstone can you take this on as part of the PoS work? |
in #1840 we're authorizing pre-genesis init-validator tx with all these validator keys:
We should use multisig to do the same for the regular init-validator tx |
we still need on-chain init-validator sigs check |
For the on-chain validator account creation and for transactions to change consensus key (pending on #36), we must also require that a signature with the consensus key is attached to the transactions to verify the key's ownership. Depends on #85 to be able to attach multiple signatures to a tx.
For this, we should update the client commands and the PoS VP to enforce this rule.
The text was updated successfully, but these errors were encountered: