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

Optimize shared secrets encryption scheme #63

Merged
merged 5 commits into from
Jan 11, 2024

Conversation

AllFi
Copy link

@AllFi AllFi commented May 8, 2023

This PR contains an implementation of the zkBob/libzeropool-zkbob#14.

In this PR the following was done:

  1. A new key kappa has been added to the Keys struct.
  2. The parameter kappa has been added to all the calls of encrypt, decrypt_out, and symcipher_decryption_keys.
  3. parse_prefix has been modified to accommodate the new Version struct.

Related PR: zkBob/libzeropool-zkbob#15
Related draft in the docs: https://app.gitbook.com/o/-Loxl4rmid_XZStTXG0U/s/-MjSwkv4zokqCUebt-98/~/changes/512/implementation/zkbob-keys/~/details

Note: Please notify me before merging this PR so I can remove the temporary patch section in Cargo.toml and update versions.

@AllFi AllFi force-pushed the feature/encryption-optimization branch from 46d0cdd to f581fb3 Compare May 11, 2023 11:09
@AllFi AllFi force-pushed the feature/encryption-optimization branch from f581fb3 to 3c21aa6 Compare May 15, 2023 09:41
@AllFi AllFi changed the title Add symmetric key, refactor code Optimize shared secrets encryption scheme May 16, 2023
@AllFi AllFi marked this pull request as ready for review May 16, 2023 10:21
Copy link
Collaborator

@EvgenKor EvgenKor left a comment

Choose a reason for hiding this comment

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

It looks good to me

libzkbob-rs-wasm/src/client/tx_parser.rs Outdated Show resolved Hide resolved
@EvgenKor EvgenKor merged commit 11c71d9 into develop Jan 11, 2024
3 checks passed
@EvgenKor EvgenKor deleted the feature/encryption-optimization branch January 11, 2024 21:11
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

Successfully merging this pull request may close these issues.

3 participants