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

sm3: add OSCCA SM3 secure hash #249

Merged
merged 1 commit into from
Jul 19, 2021
Merged

sm3: add OSCCA SM3 secure hash #249

merged 1 commit into from
Jul 19, 2021

Conversation

uudiin
Copy link
Contributor

@uudiin uudiin commented Mar 30, 2021

Add OSCCA SM3 secure hash (OSCCA GM/T 0004-2012 SM3) generic
hash transformation.

Add OSCCA SM3 secure hash (OSCCA GM/T 0004-2012 SM3) generic
hash transformation.

Signed-off-by: Tianjia Zhang <[email protected]>
@@ -0,0 +1,25 @@
[package]
name = "sm3"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like there is already a published sm3 crate:

https://crates.io/crates/sm3

The repo/source code is here:

https://github.com/andelf/rust-sca/tree/master/sm3

Perhaps we could ask the author about upstreaming it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Putting these crates on RustCrypto would be a better choice, I will try to contact the original author. Thanks.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've invited @newpavlov as an owner of sm3 on crates.io.

Feel free to overwrite it with RustCrypto crates. 😁

Also, I've invited @uudiin to rust-sca repo. You can update any READMEs and links.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow, awesome, thank you!

@tarcieri tarcieri merged commit c5f6274 into RustCrypto:master Jul 19, 2021
@tarcieri
Copy link
Member

@newpavlov can you add the RustCrypto/hashes team as an owner of the sm3 crate?

@newpavlov
Copy link
Member

@tarcieri
Done!

@tarcieri tarcieri mentioned this pull request Jul 19, 2021
@uudiin uudiin deleted the sm3 branch July 19, 2021 02:16
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.

4 participants