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.

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

ml-dsa: encode private key #892

Merged
merged 2 commits into from
Jan 26, 2025

Conversation

baloo
Copy link
Member

@baloo baloo commented Jan 24, 2025

Follow up on #891

This reworks the ml_dsa::KeyPair to keep the seed along with the SigningKey which allows to serialize the private key to the pkcs8 format after that.

@baloo baloo marked this pull request as draft January 24, 2025 00:30
@baloo baloo force-pushed the baloo/ml-dsa/encode-private-key branch 5 times, most recently from 5d051e1 to 75af08e Compare January 24, 2025 18:22
@tarcieri
Copy link
Member

@baloo can you rebase now that I've merged #893?

@baloo baloo force-pushed the baloo/ml-dsa/encode-private-key branch 2 times, most recently from 0830445 to 81d8d33 Compare January 24, 2025 21:21
@tarcieri
Copy link
Member

One more rebase and I think this is good to go

@baloo
Copy link
Member Author

baloo commented Jan 25, 2025

Yeah I’ll do that in 2-3h when I get back home.

baloo added 2 commits January 25, 2025 12:02
This ensure the keypair can only be constructed from a seed.
@baloo baloo force-pushed the baloo/ml-dsa/encode-private-key branch from 81d8d33 to 79ef8f1 Compare January 25, 2025 20:04
@baloo baloo marked this pull request as ready for review January 25, 2025 20:04
@tarcieri tarcieri merged commit e7c698a into RustCrypto:master Jan 26, 2025
4 checks passed
@baloo baloo deleted the baloo/ml-dsa/encode-private-key branch January 26, 2025 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants