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

DKG Key Refreshing via Proactive secret sharing #3

Closed
Tracked by #76
cygnusv opened this issue Oct 21, 2022 · 3 comments · Fixed by #26
Closed
Tracked by #76

DKG Key Refreshing via Proactive secret sharing #3

cygnusv opened this issue Oct 21, 2022 · 3 comments · Fixed by #26
Assignees
Milestone

Comments

@cygnusv
Copy link
Member

cygnusv commented Oct 21, 2022

Related issue: #78

@cygnusv
Copy link
Member Author

cygnusv commented Oct 28, 2022

See anoma#18

@cygnusv cygnusv self-assigned this Nov 3, 2022
@theref
Copy link

theref commented Nov 7, 2022

What would trigger this? PRE nodes currently have an idea of the fleet state. Would it just be a case of monitoring that equivalent state and triggering whenever a node leaves that was part of a DKG ceremony that I participated in?

@piotr-roslaniec
Copy link
Member

piotr-roslaniec commented Nov 18, 2022

During our code review we put forward a few ideas on how we could alter Ferveo to better fit our needs, specifically:

  • Remove weighting from nodes - each node gets an equal amount of key shares
  • Further, we set a limit of one share per node

With these changes in mind, it seems like the key refresh described in the original issue (anoma#18) could be simplified to "running the same DKG again with final key 0".

This is not exactly the kind of solution we want (Proactive secret sharing). In our approach, we would need DKG participants to construct random polynomials and use them to update their key shares.

I suspect this work may be blocked by the alternations we've made during our code review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

3 participants