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

Support Argon2id. #34

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Support Argon2id. #34

wants to merge 2 commits into from

Conversation

bryant
Copy link
Owner

@bryant bryant commented Dec 3, 2017

Fixes #27.

@coder543
Copy link

what is this PR waiting on?

FauxFaux added a commit to FauxFaux/argon2min that referenced this pull request Oct 21, 2018
@ayosec
Copy link

ayosec commented Nov 24, 2018

It seems that there is a fork in https://github.com/FauxFaux/argon2min with this patch merged.

@FauxFaux Do you plan to publish your argon2min crate?

@FauxFaux
Copy link

@ayosec: After review, I am not sufficiently confident in the code to publish it under my name, so not at this time, no.

@ayosec
Copy link

ayosec commented Nov 24, 2018

After review, I am not sufficiently confident in the code to publish it under my name, so not at this time, no.

That's fair. Thanks for the response!


For future readers: If you need Argon2id, the best (only?) option is argonautica, which uses the reference C implementation.

@est31
Copy link

est31 commented Apr 6, 2019

@FauxFaux did you have time to do such a review? I'd love to be able to use a good fast Rust based implementation of argon2.

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.

None yet

5 participants