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.

Sign up for GitHub

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

Jump to bottom

feat: add DKG verification window #1317

Merged
merged 122 commits into from
Feb 7, 2025
Merged

feat: add DKG verification window #1317

merged 122 commits into from
Feb 7, 2025

Conversation

matteojug
Copy link
Collaborator

@matteojug matteojug commented Feb 6, 2025 •
edited
Loading

Description

Part of #1300

Changes

Add dkg_verification_window, after which the signer mark the unverified dkg as failed. Also exclude failed shares from get_encrypted_dkg_shares_count.

Testing Information

Add a test to check that the shares are marked as failed and excluded from the count.

Checklist:

  • I have performed a self-review of my code
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

xoloki and others added 30 commits January 23, 2025 21:10
@xoloki
update for wsts 11
9306dfa
@xoloki
use wsts 11.0.0 from crates
c986d2a
@xoloki
fix rebase
25ac3be
@xoloki
use wsts-12.0.0 from crates
cf13953
@xoloki
add FrostCoordinator to wsts_state_machine so we can run signing roun…
6cb670a 
…ds where all signers must participate
@cylewitruk
wip
31726ee
@cylewitruk
Merge branch 'feat/frost-coordinator' into feat/require-all-signature…
63731bb 
…s-for-rotate-keys
@cylewitruk
run a dkg signing before deploying contracts
105d9a7
@cylewitruk
new wsts commit
4d406ee
@cylewitruk
wip
1218eb6
@cylewitruk
Merge branch 'main' into feat/require-all-signatures-for-rotate-keys
4179cd6
@cylewitruk
seems to work
be05f5e
@cylewitruk
working version
43501cb
@cylewitruk
Merge branch 'main' into feat/require-all-signatures-for-rotate-keys
a0ce153
@cylewitruk
remove dead code
a75f5a5
@cylewitruk
minor wsts logging tweaks
2214427
@cylewitruk
Merge branch 'main' into feat/require-all-signatures-for-rotate-keys
1511969
@cylewitruk
rename wstsmessageid variant
25d214a
@cylewitruk
it works, now need to clean up
18b69d2
@cylewitruk
downgrade wsts
7412f87
@cylewitruk
proto backwards compatability fixes
5a16608
@cylewitruk
Merge branch 'main' into feat/require-all-signatures-for-rotate-keys
0e624e6
@cylewitruk
some wsts cleanup/refactor
e867b72
@cylewitruk
think that's it
c742af2
@cylewitruk
add message support for a specific dkg wstsmessageid
347db71
@cylewitruk
add a dkg wstsmessageid variant
df5111f
@cylewitruk
merge changes from the refactor branch
e1173f6
@cylewitruk
use block hash instead of random data
758c6c2
@cylewitruk
fmt
8acccaf
@cylewitruk
remove 100% requirement for stacks signing of rotate keys
00537be
matteojug and others added 17 commits February 7, 2025 09:52
@matteojug
chore: CR nits
bf615de
@djordon @matteojug
Change the DKG shares status type.
7878be8 
Lots of small follow-up modifications
@djordon @matteojug
Update the migration query
daf4946
@djordon @matteojug
Fix up remaining queries
83b96b0
@djordon @matteojug
Oops forgot this one
ee46b3e
@djordon @matteojug
Forgot this rename
aea867b
@djordon @matteojug
Rename the new status field to
27a3bfa 
match the new column
@djordon @matteojug
Fix up the tests and add a new one
5b7cc5d
@djordon @matteojug
Change the return value of some of the
d290aa8 
queries and simplify the validation check. Also add a test.
@djordon @matteojug
Change it back
0ce46d3
@djordon @matteojug
Update the tests
3c837d8
@djordon @matteojug
Clean up the comments in the shares enum
76f6754
@djordon @matteojug
rename the rotate keys error variant
e487615
@djordon @matteojug
Use a better error variant when extracting
4ad9835 
the started_at from the state machine Id
@djordon @matteojug
Remove some of our unused error variants
21cd44b
@djordon @matteojug
Match the behavior in the in memory store
102a3d5 
with the postgres implementation
@djordon @matteojug
We do not need these errors anymore either
45c8299
Base automatically changed from 1313-validate-rotate-keys-contract-calls-with-validated-shares to feat/mock-signing February 7, 2025 09:10
cylewitruk
cylewitruk approved these changes Feb 7, 2025
View reviewed changes
Copy link
Member

@cylewitruk cylewitruk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

signer/src/storage/mod.rs Outdated Show resolved Hide resolved
signer/src/storage/postgres.rs Outdated Show resolved Hide resolved
@djordon djordon added this to the sBTC: Deposits milestone Feb 7, 2025
@matteojug matteojug changed the base branch from feat/mock-signing to main February 7, 2025 21:17
@matteojug matteojug merged commit 3564b70 into main Feb 7, 2025
8 checks passed
@matteojug matteojug deleted the feat/dkg-validity-window branch February 7, 2025 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@djordon djordon djordon approved these changes

@cylewitruk cylewitruk cylewitruk approved these changes

@MCJOHN974 MCJOHN974 MCJOHN974 left review comments

@Jiloc Jiloc Awaiting requested review from Jiloc

Assignees

@matteojug matteojug

Labels
None yet
Projects
sBTC
Status: Done
Milestone
sBTC: Deposits
Development

Successfully merging this pull request may close these issues.
6 participants
@matteojug @djordon @cylewitruk @MCJOHN974 @xoloki @Jiloc