Use BCrypt::Password instead of BCrypt::Engine #113

Open
etdsoft opened this Issue Jun 28, 2011 · 2 comments

Projects

None yet

2 participants

@etdsoft
etdsoft commented Jun 28, 2011

Hey Ryan,

It seems that BCrypt::Password.create(plaintext) would internally use BCrypt::Engine to generate a salt. Maybe that can simplify the library and have an even smaller footprint on the DB?

Maybe you have researched this and it doesn't make sense...

Here are the docs:

http://bcrypt-ruby.rubyforge.org/classes/BCrypt/Password.html#M000007

@ryanb
Owner
ryanb commented Jun 29, 2011

Yes, I believe this is what SecurePassword does. This way we don't have to store a salt. I'll mark this for being added.

@etdsoft
etdsoft commented Jun 29, 2011

Cool,

I have implemented this already. Will send a pull request just in case it makes things any easier on your side.

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