同字符串哈希不匹配：NodeJS/ReactJS改密码功能异常

我正在开发基于NodeJS、ReactJS的Web应用，希望为用户提供修改密码的功能。以下是我的路由代码：

router.put('/changepass/:user_id', auth, async (req, res) => { 
  const errors = validationResult(req); 
  if(!errors.isEmpty()) { 
    return res.status(400).json({errors: errors.array()}); 
  } 
  const {oldPassword, newPassword} = req.body; 
  try { 
    const user = await User.findById(req.params.user_id); 
    const salt = await bcrypt.genSalt(10); //10 rounds 
    const prevPassword = await bcrypt.hash(oldPassword, salt); 
    if(prevPassword !== user.password) { 
      return res.status(400).send({errors: [{msg: 'Old password does not match'}]}); 
    } 
    user.password = await bcrypt.hash(newPassword, salt); 
    await user.save(); 
  } catch(err) { 
    console.error(err.message); 
    res.status(500).send('Server Error'); 
  } 
})

问题在于，即使密码字符串相同，生成的prevPassword哈希与已存储的用户密码哈希也不匹配。为何相同密码字符串会生成两个不同的哈希？

这是bcrypt使用中非常常见的误区，核心原因是你误解了bcrypt哈希的工作机制——bcrypt每次生成哈希时都会自动生成随机的盐值，哪怕你手动指定盐，直接对比两次哈希结果也是行不通的（而且手动复用盐本身也不符合安全规范）。

具体哪里错了？ #

你当前的代码逻辑是：

1. 给用户输入的旧密码oldPassword手动生成盐，再哈希得到prevPassword
2. 把这个新生成的哈希和数据库里存储的user.password直接做字符串对比

但数据库里的用户密码哈希，是用户注册时用另一个随机盐生成的。哪怕密码明文完全相同，只要盐不一样，最终的哈希值就肯定不同，所以你的对比逻辑必然会失败。

正确的做法：用bcrypt自带的compare()方法 #

bcrypt专门提供了bcrypt.compare(plainText, storedHash)方法来解决这个问题：它会自动从存储的哈希值里提取出当初用的盐，再用这个盐去哈希输入的明文密码，最后对比两者的哈希结果是否一致。这样就能正确验证密码是否匹配了。

修改后的代码如下：

router.put('/changepass/:user_id', auth, async (req, res) => { 
  const errors = validationResult(req); 
  if(!errors.isEmpty()) { 
    return res.status(400).json({errors: errors.array()}); 
  } 
  const {oldPassword, newPassword} = req.body; 
  try { 
    const user = await User.findById(req.params.user_id); 

    // 用bcrypt.compare验证旧密码，不要自己重新哈希对比
    const isPasswordMatch = await bcrypt.compare(oldPassword, user.password);
    if(!isPasswordMatch) { 
      return res.status(400).send({errors: [{msg: 'Old password does not match'}]}); 
    } 

    // 哈希新密码时，直接传rounds参数即可，bcrypt会自动生成随机盐
    user.password = await bcrypt.hash(newPassword, 10);
    await user.save();

    // 返回成功响应，让前端知道操作完成
    res.status(200).json({msg: 'Password updated successfully'});
  } catch(err) { 
    console.error(err.message); 
    res.status(500).send('Server Error'); 
  } 
})

额外的小优化： #

• 不需要手动调用bcrypt.genSalt()，bcrypt.hash()可以直接接收rounds数值作为第二个参数，它会自动生成安全的随机盐并完成哈希
• 记得在密码修改成功后返回明确的成功响应，这样前端能更好地处理后续逻辑

内容的提问来源于stack exchange，提问作者Umair