Node.js中bcrypt.compare未触发/失效问题求助

你的代码存在三个核心问题，直接导致bcrypt密码验证功能异常：

1. bcrypt.compare参数顺序完全颠倒 #

bcrypt.compare的正确调用逻辑是先传入用户输入的明文密码，再传入数据库存储的哈希值，你当前把哈希值放在了第一个参数位置，这会直接导致验证逻辑失效，甚至回调无法按预期触发。

2. 异步回调未纳入Promise链 #

bcrypt.compare是异步操作，你在then回调里直接执行它，但没有等待它完成就返回了passwordStatus——此时passwordStatus还是初始的空字符串，后续Promise链会直接执行，完全等不到验证结果。

3. Promise链存在逻辑漏洞 #

• 验证失败分支仅设置了res.status(404)，但未返回响应或终止Promise链，导致后续then仍会执行，引发错误。
• passwordStatus的异步赋值逻辑错误，无法正确传递验证结果到后续环节。

修复后的代码 #

app.post("/login", async (req, res) => {
  const { username, password } = req.body;
  try {
    // 1. 查询用户哈希密码
    const passwordQuery = await db.query(
      "SELECT password FROM users WHERE username = $1",
      [username]
    );
    
    if (passwordQuery.rows.length === 0) {
      return res.status(404).send("用户不存在");
    }

    const hash = passwordQuery.rows[0].password;
    console.log("hash", hash);
    console.log("password", password);

    // 2. 正确执行密码验证（参数顺序：明文密码在前，哈希值在后）
    const isPasswordValid = await bcrypt.compare(password, hash);
    
    if (!isPasswordValid) {
      return res.status(401).send("密码错误");
    }

    // 3. 验证成功后获取用户ID
    const userQuery = await db.query(
      "SELECT user_id FROM users WHERE username = $1;",
      [username]
    );
    
    res.status(200).send([userQuery.rows]);
  } catch (error) {
    console.log("error:", error);
    res.status(500).send("服务器内部错误");
  }
});

关键修复说明 #

• 用async/await替代Promise链式调用，让异步逻辑更直观，避免回调嵌套和时序混乱。
• 修正bcrypt.compare的参数顺序，确保验证逻辑正确。
• 增加用户不存在的判断，提前返回响应，避免无效的后续操作。
• 每个分支都明确返回响应，避免请求挂起或错误的后续逻辑触发。
• 统一用try/catch捕获所有异常，确保错误能被正确处理并反馈给客户端。

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