Skip to content
This repository has been archived by the owner on Dec 12, 2021. It is now read-only.

Replace BCrypt::Engine with BCrypt::Password #117

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

Replace BCrypt::Engine with BCrypt::Password #117

wants to merge 5 commits into from

Commits on Jul 9, 2011

  1. No need to use Engine::generate_salt 

    Instead of using Engine::generate_salt and Engine::hash_secret, we can replace
them with Password.create
    Daniel Martin committed Jul 9, 2011
    Configuration menu
    Copy the full SHA
    1a59add View commit details
    Browse the repository at this point in the history

  2. No need set the salt separately in the fixtures

    Daniel Martin committed Jul 9, 2011
    Configuration menu
    Copy the full SHA
    41d51a2 View commit details
    Browse the repository at this point in the history

  3. Amend the test cases: we no longer generate a salt

    Daniel Martin committed Jul 9, 2011
    Configuration menu
    Copy the full SHA
    2c91c0d View commit details
    Browse the repository at this point in the history

  4. Reduce the number of fields in the users table 

    BCrypt::Password handles the salt directly so we don't need to store it in a
separate column.
    Daniel Martin committed Jul 9, 2011
    Configuration menu
    Copy the full SHA
    e7b03df View commit details
    Browse the repository at this point in the history

  5. Update the .gemspec with the new version of bcrypt

    Daniel Martin committed Jul 9, 2011
    Configuration menu
    Copy the full SHA
    bcb525f View commit details
    Browse the repository at this point in the history