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

Signing key management #30

Closed
spengrah opened this issue Apr 16, 2020 · 3 comments · Fixed by #146
Closed

Signing key management #30

spengrah opened this issue Apr 16, 2020 · 3 comments · Fixed by #146
Assignees
Labels
Projects
Milestone

Comments

@spengrah
Copy link
Member

spengrah commented Apr 16, 2020

Users need a way to store their MACI message-signing keys

3box? Ceramic?

@spengrah spengrah transferred this issue from clrfund/web Apr 21, 2020
@spengrah spengrah moved this from MVP must-have: To do to PoC: To do in MVP Jun 17, 2020
@xuhcc xuhcc added this to the Milestone 3 milestone Aug 24, 2020
@xuhcc
Copy link
Contributor

xuhcc commented Oct 3, 2020

We can simply keep the encrypted EdDSA keys in local storage of the browser. Keys can be discarded when the funding round ends.

3box option is interesting but it is needed only if there are users who access dapp from multiple devices.

@xuhcc xuhcc self-assigned this Oct 3, 2020
@xuhcc xuhcc moved this from TODO to In progress in MVP Oct 3, 2020
@xuhcc xuhcc added the frontend label Oct 3, 2020
@auryn-macmillan
Copy link
Member

@xuhcc for milestone 3, I assume we're just rolling with the current local storage solution?

@xuhcc
Copy link
Contributor

xuhcc commented Oct 13, 2020

@auryn-macmillan Yes, for now we will store the encrypted key in local storage. The dapp asks user to sign a message at log in. This signature is then used to create a secret key which encrypts the data using AES.

I'll publish the code soon for review.

xuhcc added a commit to xuhcc/clrfund that referenced this issue Oct 14, 2020
MVP automation moved this from In progress to Done Oct 16, 2020
daodesigner pushed a commit that referenced this issue Sep 19, 2021
Add recipient transaction on form submission
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
MVP
  
Done
Development

Successfully merging a pull request may close this issue.

3 participants