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

[wip] Native verification #583

Open
wants to merge 14 commits into
base: develop
from
Open

[wip] Native verification #583

wants to merge 14 commits into from

Conversation

@dark64
Copy link
Collaborator

dark64 commented Mar 19, 2020

Work in progress.

  • Added verify command to the CLI
  • Implemented native verification for groth16 (utilizing bellman)
  • Simplified parsing of verification key by adding a key-value parser.
  • Added --raw flag to setup and generate-proof commands
  • Changed base64 encoding to hex encoding

Note: This PR had to change verification key by adding an extra key called vk.raw that contains raw hex encoded bytes representing bellman's VerifiyingKey object. Same applies with the proof which contains raw format of Proof. This only happens if raw flag is passed.

@dark64 dark64 requested a review from Schaeff Mar 23, 2020
@dark64 dark64 changed the title |wip] Native verification [wip] Native verification Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.