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

Do not describe recovery data as "public" and mention deriving an encryption key from the deed. #39

Open
jonasnick opened this issue Jul 19, 2024 · 1 comment

Comments

@jonasnick
Copy link
Collaborator

In the design section, it currently says:

Simple backups: The capability of ChillDKG to recover devices from a static seed and public recovery data avoids the need for secret per-session backups, enhancing user experience.

If the recovery data is public it might reveal sensitive information. We mention that clearly about that in the later sections, but ideally we would have a less misleading phrasing here.

Additionally, we should consider mentioning that it's possible to derive an encryption key from the seed (and maybe how to do that), such that recovery from encrypted recovery data requires no additional secret besides the seed.

@real-or-random
Copy link
Collaborator

real-or-random commented Sep 26, 2024

Additionally, we should consider mentioning that it's possible to derive an encryption key from the seed (and maybe how to do that), such that recovery from encrypted recovery data requires no additional secret besides the seed.

We could consider just adding an individual encryption/decryption method as part of the spec. When we had discussed this earlier, I was hesitating to suggest this due to scope creep. But symmetric encryption is just a few lines of code, so perhaps it's not too much overhead in terms of complexity, and it will save others the work to come up with their own method encryption (and agree on it).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants