-
Notifications
You must be signed in to change notification settings - Fork 230
x448: initial import #1250
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
base: master
Are you sure you want to change the base?
x448: initial import #1250
Conversation
48849e0
to
c67e30d
Compare
c67e30d
to
8675b5f
Compare
This kind of broke when we merged #1284 cc @daxpedda I'm confused. I don't understand how https://www.rfc-editor.org/rfc/rfc7748#section-5 asks for the scalar to be a 56-bytes string and now the Am I supposed to use the |
Apologies, I noticed indeed that this should have been Feel free to fix this in this PR. |
No worries, I'm not sure I'm following what you're saying, I guess I'll wait for your PR ^^ |
89ee8ef
to
2280752
Compare
2280752
to
7368a67
Compare
875180d
to
f8cd31a
Compare
I have a new and improved PR up for |
42c40da
to
9138194
Compare
@baloo is there a specific reason this is still draft? It'd be good to get it landed so we make sure it gets updated when |
36aa271
to
87d027d
Compare
I struggled for a bit with various I'll keep rebasing and testing along with the merges until we can get this out of draft :) |
a9bb6e9
to
0c69dd6
Compare
I guess the scalar wasn't that hard to figure out in the end. I've sent a PR over to @daxpedda 's branch to migrate over to |
All of the original authors have agreed to make this change
0c69dd6
to
7f10832
Compare
This imports https://crates.io/crates/x448 in https://github.com/RustCrypto/elliptic-curves/