-
Notifications
You must be signed in to change notification settings - Fork 256
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
[zk-sdk] Change ZeroBalanceProof
to ZeroCiphertextProof
#1085
[zk-sdk] Change ZeroBalanceProof
to ZeroCiphertextProof
#1085
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1085 +/- ##
=========================================
Coverage 82.2% 82.2%
=========================================
Files 868 880 +12
Lines 234388 235584 +1196
=========================================
+ Hits 192668 193689 +1021
- Misses 41720 41895 +175 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! There are also references to "zero-balance" in transcript.rs:
agave/zk-sdk/src/transcript.rs
Line 37 in 6398ddf
/// Append a domain separator for zero-balance proof. |
Thanks for the review! Yes, I forgot to mention it in the PR description. There are some other things to address relating to transcript, so there will be a follow-up PR addressing those and fixing the zero-balance name there. |
Problem
The
ZeroBalanceProof
certifies that a ciphertext is an encryption of the number zero. It does have applications to private transfers, but the name is unnecessarily specific to this application only. It makes sense to rename the type to something more general as proposed in #671.Summary of Changes
Rename
ZeroBalanceProof
toZeroCiphertextProof
.Fixes #