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

add blake #46

Closed
wants to merge 6 commits into from
Closed

add blake #46

wants to merge 6 commits into from

Conversation

kazcw
Copy link
Contributor

@kazcw kazcw commented Dec 1, 2017

I wrote a rust implementation of the "Blake" SHA3 finalist, based closely on the reference implementation. It's a little raw; it needs docs, lots more tests, and a macro cleanup I think. But it's getting the right results (I've tested it on a lot more random inputs than are currently committed to repo), it performs fine, and I think I've implemented the same API as the other crates in this workspace.

@newpavlov
Copy link
Member

Thank you for your contributions!

Unfortunately it will be some time before I will be able to check your PRs. I'll try to do it in December!

@kazcw
Copy link
Contributor Author

kazcw commented Dec 5, 2017

Thank you for all the work you're doing on this project! Unifying around thoughtfully-designed APIs is so important for building up a pure Rust crypto ecosystem.

@newpavlov newpavlov mentioned this pull request Feb 17, 2018
18 tasks
@kazcw
Copy link
Contributor Author

kazcw commented Feb 1, 2019

I no longer have time to keep synchronizing this PR with the published crate. BLAKE can be found here: https://github.com/cryptocorrosion/cryptocorrosion/tree/master/hashes/blake

@kazcw kazcw closed this Feb 1, 2019
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