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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

new encryption + more hashes 馃惓 #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

new encryption + more hashes 馃惓 #15

wants to merge 1 commit into from

Conversation

3lLobo
Copy link
Owner

@3lLobo 3lLobo commented Oct 3, 2022

Changes:

  • use Metamask only to encrypt a key
  • this key is the TOTP secret -> we can update the tree without the user having to re-scan the qr
  • same key used to en/decrypt the merkle tree
  • using crypto-js and AES for en/decryption
  • rewrote and parametized the prepareMerkleTree() function.
  • totp exponent set to 16 which generates hashes for 22 days into the future.
  • generation takes approx 3 sec on my machine.
  • TODO: adjust the OtpMerkleTreeVerifier contract to accept more hashes for validation.

@3lLobo 3lLobo added the frontend the pretty part label Oct 3, 2022
@3lLobo 3lLobo self-assigned this Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danieljcss danieljcss Awaiting requested review from danieljcss

@rishotics rishotics Awaiting requested review from rishotics

Assignees

@3lLobo 3lLobo

Labels
frontend the pretty part
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@3lLobo