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

bumped curv-kzen to 0.10 + Upgraded code to compile #214

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

Conversation

gdnathan
Copy link

@gdnathan gdnathan commented Aug 1, 2023

Bumped curv-kzen to 0.10.

Notable changes to files:

  • Now that VerifiableSS takes a second generic parameter H: Digest + Clone, I used sha2::Sha256 as this generic parameter, as the curv-kzen lib does
  • VerifiableSS also takes a new "proof" field. As there is no default, I copied the logic from one of the files of curv-kzen and did this:
    let witness = Scalar::random();
    let proof = DLogProof::<Secp256k1, Sha256>::prove(&witness);
  • As Sha256 implement neither Serialize nor Deserialize, some of the tests doesn't pass anymore. I commented them out until we find a solution. If you have any required change, or any advice on how to fix the deserialize/serialize issue, please tell me.

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.

1 participant