12

我正在尝试使用 Sinatra 和 BCrypt 实现看似非常简单的身份验证方法，但显然我遗漏了一些东西......

用户被预先分配了一个临时密码，该密码以明文形式存储在数据库中。

我对临时密码进行身份验证，然后创建一个 salt 和 password_hash 并将它们作为字符串写入数据库（在本例中为 mongo）。

为了进行身份验证，我从数据库中获取盐和用户密码进行比较。

post "/password_reset" do
  user = User.first(:email => params[:email], :temp_password => params[:temp_password])
  if dealer != nil then
  password_salt = BCrypt::Engine.generate_salt
  password_hash = BCrypt::Engine.hash_secret(params[:password], password_salt)
  user.set(:password_hash => password_hash)
  user.set(:password_salt => password_salt)
  end
end

post "/auth" do
  @user = User.first(:email => params[:email])
  @user_hash = BCrypt::Password.new(@user.password_hash) #because the password_hash is  stored in the db as a string, I cast it as a BCrypt::Password for comparison
  if @user_hash == BCrypt::Engine.hash_secret(params[:password], @user.password_salt.to_s)   then
    auth = true
  else
    auth = false
  end
end

BCrypt::Engine.hash_secret(params[:password], password_salt) 返回的值与存储在数据库中的值不同（两者都属于 BCrypt::Password 类，但它们不匹配）。

我在这里想念什么？非常感谢您的任何见解！

马克

4

1 回答 1

24

BCrypt::Password是 的子类String，它重写了该==方法以使检查密码更容易。当你这样做

if @user_hash == BCrypt::Engine.hash_secret(params[:password], @user.password_salt.to_s)

您最终执行了两次哈希，因此它们不匹配。如果您直接@user.password_hash比较而不是使用BCrypt::Password.new，您应该会看到它们匹配。

使用 bcrypt-ruby 作为密码更“正确”的方法是根本不使用Engine类，只使用Password类。您不需要自己管理盐，bcrypt 会处理它并将其包含在密码哈希字符串中：

password_salt = BCrypt::Engine.generate_salt
password_hash = BCrypt::Engine.hash_secret("s3kr1t!", password_salt)

puts password_salt
puts password_hash

产生这样的东西：

$2a$10$4H0VpZjyQO9SoAGdfEB5j.
$2a$10$4H0VpZjyQO9SoAGdfEB5j.oanIOc4zp3jsdTra02SkdmhAVpGK8Z6

如果你运行它，你会得到一些稍微不同的东西，因为会生成不同的盐，但你可以看到密码散列包括盐。

在你的情况下，你想要这样的东西：

post "/password_reset" do
  user = User.first(:email => params[:email], :temp_password => params[:temp_password])
  if dealer != nil then
    password_hash = BCrypt::Password.create(params[:password])
    user.set(:password_hash => password_hash) # no need to store the salt separately in the database
  end
end

post "/auth" do
  @user = User.first(:email => params[:email])
  @user_hash = BCrypt::Password.new(@user.password_hash)
  if @user_hash == params[:password]  then # overridden == method performs hashing for us
    auth = true
  else
    auth = false
  end
end
于 2012-08-19T19:29:28.283 回答