-
Notifications
You must be signed in to change notification settings - Fork 70
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
Refactor 2PC GHASH #171
Refactor 2PC GHASH #171
Conversation
5131145
to
5ad6301
Compare
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.
Nice 🚀, ACK with some small changes
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.
ack
#[derive(Debug, Clone, Builder)] | ||
pub struct GhashConfig { | ||
/// Initial number of block shares to provision | ||
#[builder(default = "1024")] |
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.
Don't we want 1026 here and below ?
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 like powers of 2 🙃 but if we do find ourselves hitting against this limit we can adjust the default
This PR moves the ghash impl into the
universal-hash
crate. The core logic should be untouched.Changes
UniversalHash
trait which is a common interface between GHASH and Poly1305max_message_length
->max_block_count
as the message vs block naming was a bit confusing.