Node.bcrypt.js 中哈希值不一致问题的分析与解决

问题现象

在使用Node.bcrypt.js进行密码哈希处理时，开发者遇到了一个奇怪的现象：当使用bcrypt.hash()方法生成哈希值后，直接将该值赋给用户对象的密码属性并保存到数据库时，最终存储的哈希值与最初生成的哈希值不一致。具体表现为：

1. 控制台打印的初始哈希值：$2b$10$8UMFhEZrwc85vytIGyE2v.mX3CYU/ANojK8ZeaO2bXPs5Lttis7sq
2. 实际存入数据库的哈希值：$2b$10$ELPnmKkLLvrPW7QCVSrIp.c3Jn5VuoTsEL7OKnJ5QnJe6ZC2jYb6q

根本原因分析

经过深入研究，发现问题源于Mongoose的中间件(pre-save hook)机制。当开发者使用以下代码时：

userSchema.pre('save', async function () {
    const salt = await bcrypt.genSalt(10)
    if (!this.password) return
    this.password = await bcrypt.hash(this.password, salt)
})

这个预保存钩子会在每次调用user.save()方法时自动执行。这意味着：

1. 开发者手动调用bcrypt.hash()生成哈希值并赋给user.password
2. 然后调用user.save()保存用户
3. Mongoose在保存前再次触发pre-save钩子
4. 钩子中又对密码进行了重新哈希处理
5. 最终存入数据库的是第二次哈希的结果

解决方案

针对这个问题，有以下几种解决方案：

方案一：禁用预保存钩子

在需要手动处理密码哈希的场景下，可以临时禁用预保存钩子：

user.password = await bcrypt.hash(password, salt)
await user.save({ validateBeforeSave: false })

方案二：修改预保存钩子逻辑

改进预保存钩子，使其只在密码被修改时才重新哈希：

userSchema.pre('save', async function () {
    if (!this.isModified('password')) return
    
    const salt = await bcrypt.genSalt(10)
    this.password = await bcrypt.hash(this.password, salt)
})

方案三：分离密码哈希逻辑

将密码哈希逻辑提取为单独的方法，避免依赖预保存钩子：

userSchema.methods.hashPassword = async function(password) {
    const salt = await bcrypt.genSalt(10)
    return await bcrypt.hash(password, salt)
}

// 使用方式
user.password = await user.hashPassword(password)
await user.save()

最佳实践建议

1. 一致性处理：在整个应用中统一使用一种密码处理方式，要么全部使用预保存钩子，要么全部手动处理。

2. 密码修改检测：在预保存钩子中始终检查isModified('password')，避免不必要的重新哈希。

3. 错误处理：添加适当的错误处理逻辑，确保哈希过程中的任何错误都能被捕获和处理。

4. 性能考虑：bcrypt的哈希操作是CPU密集型操作，应避免在不需要的情况下重复执行。

5. 测试验证：编写单元测试验证密码哈希和验证流程，确保行为符合预期。

总结

Node.bcrypt.js作为Node.js生态中广泛使用的密码哈希库，其安全性依赖于正确的使用方式。理解Mongoose中间件机制与bcrypt的交互方式，能够帮助开发者避免这类哈希值不一致的问题。通过采用上述解决方案中的任何一种，都可以确保密码哈希过程的一致性和可靠性，从而保障用户认证系统的安全性。