-
Notifications
You must be signed in to change notification settings - Fork 47
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
Rewrite KOS15 #37
Rewrite KOS15 #37
Conversation
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.
🚀 Very clean and nice. Have some comments...
@@ -0,0 +1,85 @@ | |||
use derive_builder::Builder; | |||
|
|||
/// KOS15 sender configuration. |
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.
Maybe a sentence explaining why Sender
and Receiver
each have sender_commit
and receiver_commit
.
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.
I removed the unused configurations
ot/mpz-ot/src/kos/mod.rs
Outdated
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.
Can we have some tests for check
and verify
failures, for the KOS check and KOS committed OT?
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.
This is tested in the core crate
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.
I have not fully reviewed yet since some of my feedback depends on the things we are discussing on discord.
I'll review again when we agree on the final design.
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.
gw!!!
* refactor kos15 * remove panic * fix derandomize * fix --no-default-features, and unused import * rebase fixes * sample delta with cointoss * remove unused config * update committed sender comments * PR comments * PR changes * add minimum extend check * decouple PayloadRecord * final PR changes
* refactor kos15 * remove panic * fix derandomize * fix --no-default-features, and unused import * rebase fixes * sample delta with cointoss * remove unused config * update committed sender comments * PR comments * PR changes * add minimum extend check * decouple PayloadRecord * final PR changes
This PR reimplements KOS15, fixing some security and performance issues. There might be a couple odds and ends that still need to be buttoned up, but I wanted to open this now and start getting feedback sooner.
Changes
delta
used by the sender in the setup.