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 password salts and encrypt with bcrypt #19

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
3 participants
@deepwell
Copy link

deepwell commented Dec 29, 2012

Patch to add password salts, and to make them secure unlike the default md5/sha1 options. Simpler approach taken compared to other pull requests asking for the same thing.

md5, sha1, and many other encryption algorithms should not be used to encrypt passwords as they are no longer considered secure.

@karlingen

This comment has been minimized.

Copy link

karlingen commented Jul 31, 2013

@deepwell
Is this still applicable? If yes, why is it not implemented yet?

@deepwell

This comment has been minimized.

Copy link
Author

deepwell commented Aug 1, 2013

Yes. I implemented it, just waiting for it to be merged.

@KerryRitter

This comment has been minimized.

Copy link

KerryRitter commented Nov 14, 2013

Has this been merged yet? If so, do implement do I just change the 'hash' config setting to "bcrypt"?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.