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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Implement a RedJubjubVerifier service #407

Closed
dconnolly opened this issue May 28, 2020 · 1 comment · Fixed by #460
Closed

Implement a RedJubjubVerifier service #407

dconnolly opened this issue May 28, 2020 · 1 comment · Fixed by #460
Assignees
@dconnolly
Labels
NU-1 Sapling Network Upgrade: Sapling specific tasks
Milestone
Transaction Valid...

Comments

@dconnolly
Copy link
Contributor

Part of #196
@dconnolly dconnolly added Poll::Ready NU-1 Sapling Network Upgrade: Sapling specific tasks labels May 28, 2020
@dconnolly dconnolly added this to the Validate transactions. milestone May 28, 2020
@dconnolly dconnolly self-assigned this May 28, 2020
@dconnolly dconnolly changed the title Implement a RedJubjubVerifier service Implement a RedJubjubVerifier service May 28, 2020
@dconnolly
Copy link
Contributor Author

Related: ZcashFoundation/redjubjub#30

@dconnolly dconnolly linked a pull request Jun 10, 2020 that will close this issue
Redjubjub signature verifier service #460
Merged
@dconnolly dconnolly mentioned this issue Jun 25, 2020
Closed
dconnolly added a commit that referenced this issue Jul 3, 2020
@dconnolly
Update to use merged redjubjub::batch::Verifier
cf1dcd1 
Now the Verifier is agnostic of redjubjub SigTypes.  Updated tests to
generate sigs of both types and batch verifies the whole batch.

Resolves #407
dconnolly added a commit that referenced this issue Jul 3, 2020
@dconnolly
Update to use merged redjubjub::batch::Verifier
f6301df 
Now the Verifier is agnostic of redjubjub SigTypes.  Updated tests to
generate sigs of both types and batch verifies the whole batch.

Resolves #407
dconnolly added a commit that referenced this issue Jul 8, 2020
@dconnolly
Update to use merged redjubjub::batch::Verifier
fd685c1 
Now the Verifier is agnostic of redjubjub SigTypes.  Updated tests to
generate sigs of both types and batch verifies the whole batch.

Resolves #407
dconnolly added a commit that referenced this issue Jul 8, 2020
@dconnolly
Add RedJubjub signature verifier service (#460)
2cd58c8 
Using tower-batch-based async pattern.

Now the Verifier is agnostic of redjubjub SigTypes.  Updated tests to
generate sigs of both types and batch verifies the whole batch.

Resolves #407
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dconnolly dconnolly

Labels
NU-1 Sapling Network Upgrade: Sapling specific tasks
Projects
None yet
Milestone
Transaction Validation
Development

Successfully merging a pull request may close this issue.

Redjubjub signature verifier service ZcashFoundation/zebra
1 participant
@dconnolly