0

我似乎在努力处理我的应用程序中的任何数据库错误。我有一个更新用户密码的简单查询

// Update Users Password
async function updatePassword(password, id) {
  const client = await pool.connect();
  const sql = format('UPDATE users SET password = $1 WHERE id = $2');
  try {
    await pool.query(sql, [password, id]);
  } catch (e) {
    console.error('Error Occurred', e);
    client.release();
  }
}

服务器端我的 routes.js 中有这个

const newPassword = bcrypt.hashSync(req.body.update_password, 10);
try {
  await queries.updatePassword(newPassword, req.body.user_id);
  res.status(200).send({ result: 'Password Updated Successfully' });
} catch (e) {
  res.status(400).send({ result: 'Oops, please try again' });
}

如果没有数据库错误，那么这将按预期工作，但是如果例如我提供一个字符串作为我的 id（强制错误），那么我会按预期将此错误打印到控制台

Error Occurred { error: invalid input syntax for integer: "string"

但仍然res.status(200).send({ result: 'Password Updated Successfully' });发送。

记录为

POST /update_password 200

我如何捕捉错误，因为我认为我在 try/catch 块中？

谢谢

4

1 回答 1

1

您的问题与您使用的库无关。您只是滥用async/await错误报告。

您的函数updatePassword不会重新引发错误，从而将其吞下。让它重新抛出错误，错误将正确报告：

catch (e) {
    console.error('Error Occurred', e);
    throw e;
  }

至于数据库代码，它看起来也很糟糕——你在方法内部分配了一个新连接，但只在内部释放它catch，这意味着在没有抛出错误的情况下，连接将保持占用状态，从而很快耗尽连接池。

于 2018-11-02T03:59:28.440 回答