flow-core-contracts icon indicating copy to clipboard operation
flow-core-contracts copied to clipboard

Published 20 hours ago • verified publisher icon onflow

WIP: Adds Proof of Possession verification during node registration

Open joshuahannan opened this issue 3 years ago • 2 comments

https://github.com/dapperlabs/flow-go/issues/6284

Adds a parameter to the addNodeRecord method in the staking contract for the proof of possession. In the NodeRecord init function, this is verified with the staking key.

Also updates the locked tokens contract, staking collection, and all of their transactions to include this parameter

joshuahannan avatar Jul 26 '22 20:07 joshuahannan

I've started to add the tests on top of your work, but eventually hit some issues with the contracts. They issues should be easier for you to solve. My work is on this PR (against your branch), feel free to review it and merge it.

tarakby avatar Aug 08 '22 20:08 tarakby

This PR is ready but will wait for the Ledger integration to be merged. This would simplify the contract updates during the future network sporks, while Ledger updates are being developed.

tarakby avatar Aug 19 '22 20:08 tarakby

Next step for this PR is to rebase on the stable Cadence branch

tarakby avatar May 03 '24 20:05 tarakby

Rebasing the PR on the stable-cadence branch is done in separate PR https://github.com/onflow/flow-core-contracts/pull/425. The current PR won't be merged to Cadence v0 and will be used directly with Cadence v1

tarakby avatar May 07 '24 09:05 tarakby