DKG v3.0.0 #158
Merged
DKG v3.0.0 #158
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* aligment with new specs * fix ssz generation * align output in gwei according new eth2 package * remove duplicated crypto functions * update spec * align with latest spec * local json encoding * change it ssvlabs * switch to correct spec * update json marshalling at wire (#99) * lint --------- Co-authored-by: Pavel Krolevets <[email protected]>
main 2 unstable
* initial resigning * add routes + flow * working resigning with result validation * fix tests * add integration tests for resiging * update integration tests to use cli commands * initial spec alignment + reshare module * working resharing * add EIP1271 owner signature verification * update resharing + add docker demo for resharing * add reshare threshold + integration tests * add threshold integration tests * fix logic at message processing * add eip1271 sig integration tests * add eip1271 sig verification to resign + integration/docker tests * add eip1271 sig resign test + bulk tests * fix/update unit + integration tests * update tests + fix bls set * PR comments resolved #1 * PR comments resolved #2 * PR comments resolved #3 * PR comments resolved #4 * PR comments resolved #5 * PR comments resolved #6 * fix tests + docker build update * update CI * fix test * lint * add more reshare integration tests * go 1.23 * add unit tests for join/disjoin sets of operators * lint * add proofs validation before sending + more reshare threshold integration tests * fix unit tests * lint * fix review comments * remove typo and unused struct --------- Co-authored-by: Matus Kysel <[email protected]>
* fix traingling / on ping * add current version to keyshare * Put correct config format to cli flags
* add refactor from past * refactor resign & reshare * fix critics
* rename to ssvlabs * fix the tests
* copyright pavel * fix typo * readme update --------- Co-authored-by: Matus Kysel <[email protected]>
fix amount marshaling and keyshare version
* add filepath.IsLocal checks to path flags * fix tests
* add gnosis js examples * add multisig support * update on-chain gnosis example * add off-chain multisig gnosis example * add support to multisig wallets by seperating the stage of submitting resign/reshare message and submitting signature * update reshare bulk marshaling * update operator handlers * update integration test * update Reshare and Resign hash from hash of tree root to hash of marshalled bytes * working json bulk resign/reshare decoding * fix bugs in hashing messages and initiating operator resign/reshare messages map * fix operators marshalling * Update test * simplify code * add error handling * revert changes on operator side to process resign/reshare message and its signature at the same stage * udpate initiator inputs to start resign/reshare to pass in signatures instead of secret keys * update test * implement bulk resign and resshare * update cli to support bulk resign/reshare * add clis for generating resign/reshare messages * remove a comment * fix bugs and update tests * change nonces input to a single nonce * update cli to support proofs as a string * fix bugs in cli and update tests * update tests * update tests * fix wrong formatting bug in writing resign/reshare messages to json file * fix spellings * fix spellings * add bulk reshare tests * update docker for generating resign/reshare messages * fix comments * remove unused function * fix comments * update variable naming in multisig tests * lint * increase timeout for tests * relax path requirements * fix tests --------- Co-authored-by: pavelkrolevets <[email protected]> Co-authored-by: Matus Kysel <[email protected]>
* add multisig check to health_check * lint * update message * add ping test * update test * minor fixes
* change rate limits and instance time to mitigate ddos * fix instance test * relax request limita to routes * increase test time * lint
…VE_BALANCE (#138) * pectra ready * Update integration_test/init_bulk_test.go Co-authored-by: Matus Kysel <[email protected]> * Update integration_test/init_bulk_test.go Co-authored-by: Matus Kysel <[email protected]> * fix amount af verify * fix operator tests * comment out some 100 tests --------- Co-authored-by: Matus Kysel <[email protected]>
* add json marshal/unmarshal for resign/reshare json * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * Update pkgs/wire/types_json.go Co-authored-by: Matus Kysel <[email protected]> * comment out test with changed json encoding --------- Co-authored-by: Matus Kysel <[email protected]>
* fix generation command * review fixes * fix the tests
* change message hash to align with Gnosis SDK for multisig * add one more Metamask sig example * finalize Metamask example * update react example * update react * add log of message hash to operator * fix tests * critic
* fix amount check at deposit validate lost after merges * fix tests * update tests * update tests
* refactor flags + add tlsInsecure flag + change default TLS config * update docker config + add default CA cert path /etc/ssl/certs/ca-certificates.crt * lint * rename filenames at /flags
* add messages to ping/init/reshare * make ssz errors more readable * wrap errors and remove panic at parsing ssz error * add error for incorrect offset to pong message * lint * minor fix
* fix audit issue 5.3 Crash by Malicious Operator * add unit test for instance check * add default
* add support for owner change * add test and fix owner change * fix more tests
* add broadcast for response and justification phases * update message + refactor * lint
MatusKysel
requested review from
pavelkrolevets,
moshe-blox,
GalRogozinski and
alan-ssvlabs
* return fix for Arbitrary Path Access * fix docker to use local paths (#160) --------- Co-authored-by: Pavel Krolevets <[email protected]>
pavelkrolevets
approved these changes
Nov 19, 2024
GalRogozinski
approved these changes
Nov 19, 2024
alan-ssvlabs
approved these changes
Nov 19, 2024
… IDs (#162) * Complete fix of 5.4 Reshare Denial-of-Service via Predicable Instance IDs * fix tests
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.