Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants