升级至Node 8.10.0后密码哈希输出不一致问题求助

你碰到的这个问题完全是Node.js 8.x版本中Buffer模块行为变更导致的，核心问题出在你代码里的第二行new Buffer(this.salt, 'base64')，第一行的冗余转换虽然不直接引发问题，但也可以优化。

问题根源分析 #

先拆解两行代码的问题：

1. 第一行代码：this.salt = Buffer.from(crypto.randomBytes(16).toString('base64'), 'base64');
   这行其实是做了无用功——crypto.randomBytes(16)已经返回一个Buffer，转成base64字符串再转回到Buffer，结果和原Buffer完全一致。这行本身不会导致哈希不一致，但属于冗余操作。

2. 第二行代码：return crypto.pbkdf2Sync(password, new Buffer(this.salt, 'base64'), 10000, 64, 'sha1').toString('base64');
   这才是罪魁祸首：

   • 在Node.js 6.9.x中，当你用new Buffer(buffer, encoding)时，如果第一个参数是Buffer，Node.js会隐式把Buffer转成utf8字符串，再用base64编码去解码这个字符串。这其实是一个未被严格限制的错误用法，但当时没有抛出警告。
   • 升级到8.10.0后，Node.js修复了这个行为：如果第一个参数是Buffer，new Buffer()会直接复制该Buffer，完全忽略第二个'base64'参数。这就导致传递给pbkdf2Sync的盐值和旧版本完全不同，生成的哈希自然无法匹配旧密码。

解决方案 #

1. 修复新密码的哈希生成逻辑

首先修改代码，确保盐值处理逻辑正确：

// 简化盐值生成：直接用randomBytes返回的Buffer
this.salt = crypto.randomBytes(16);

// 生成哈希时直接传入盐值Buffer，不需要再转换
return crypto.pbkdf2Sync(password, this.salt, 10000, 64, 'sha1').toString('base64');

2. 兼容旧用户的密码验证

为了让已经用旧逻辑生成哈希的用户能正常登录，你需要在验证时同时支持新旧两种逻辑：

function verifyPassword(inputPassword, storedHash, storedSalt) {
  // 先尝试用新逻辑验证
  const newHash = crypto.pbkdf2Sync(inputPassword, storedSalt, 10000, 64, 'sha1').toString('base64');
  if (newHash === storedHash) {
    return true;
  }

  // 新逻辑失败，用旧逻辑模拟6.x的行为验证
  const oldSaltBuffer = new Buffer(storedSalt.toString('utf8'), 'base64');
  const oldHash = crypto.pbkdf2Sync(inputPassword, oldSaltBuffer, 10000, 64, 'sha1').toString('base64');
  return oldHash === storedHash;
}

这样既保证新用户的密码用正确逻辑生成，也不影响旧用户登录。

3. 彻底移除旧的Buffer用法

另外，建议全面替换new Buffer()为Buffer.from()，因为Node.js已经废弃了前者的不安全用法，Buffer.from()的行为更明确、更安全。

总结 #

这个问题本质是Node.js对Buffer构造函数的行为修复，从“兼容错误用法”转向“严格遵循参数语义”。修复后不仅解决了登录问题，还能避免旧用法带来的潜在安全风险。

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