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 ops and mem limits to pwhash functions #35

Open
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@ECrownofFire

ECrownofFire commented Oct 27, 2018

With these commits, the pwhash family of functions can accept either atoms (interactive, moderate, or sensitive) or positive integers (within crypto_pwhash_(OPS/MEM)LIMIT_(MAX/MIN)).

The versions that don't take limits are maintained, and simply default to interactive/interactive as they did before.

@ECrownofFire ECrownofFire force-pushed the ECrownofFire:master branch from a26b216 to 8c0917b Oct 27, 2018

ECrownofFire added some commits Oct 27, 2018

Add choice of ops and mem limits to pwhash_str
It natively checks atoms, which is kinda messy, but it avoids having to
export the libsodium pwhash constants, which is nice.

@ECrownofFire ECrownofFire force-pushed the ECrownofFire:master branch from 8c0917b to 4afa6fc Oct 28, 2018

@jlouis

This comment has been minimized.

Owner

jlouis commented Nov 8, 2018

Nice work, I'll have a look and get it in over the next couple of days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment