await async - ESLint require-atomic-updates 中的竞争条件错误

Question

下面的代码在使用ESLint 进行检查时会导致竞争条件：

  let match = false

  try {
    match = await something()
  } catch (err) {
    // do something
  }
  if (match === false) {
    // do something
  }

编写这段代码的更好方法是什么？

编辑：

  let request = ctx.request.body || {}
  let password = request.password
  let match = false

  try {
    match = await bcrypt.compare(password, 'xxxxx')
  } catch (err) {
    ctx.throw(401, err)
  }
  if (match === false) {
    ctx.throw(401, 'invalid password')
  }

  ctx.body = {
    message: 'logged in ok'
  }

来自 ESLint 的错误：

可能的竞争条件：ctx.body 可能会根据 ctx.body require-atomic-updates 的过时值

Answer 1

您可以放心地忽略警告:)

ESLint 是为了捕捉这样的东西：

 let value = 0;

async function race() {
  value += await Promise.resolve(1);
  console.log(value);
}

race(); race();

在这种情况下，race 在堆栈上记忆 value，awaits 一个记号，然后写回value。由于其他代码同时运行，value 可能已被更改，然后更新可能会关闭......它不是原子的。

但是，在您的情况下，您从ctx.request.body 读取并写入ctx.body，因此没有非原子更新。此外，可能没有其他中间件同时访问相同的ctx，因此不能有任何并发​​修改。因此，在您的情况下，这是一个误报，甚至可以怀疑这是否是肯定的（这可能是 ESLint 中的一个错误）。

Answer 2

我意识到这个答案有点晚了，但对于任何遇到这个问题的未来用户来说，要禁用此规则，在您的 .eslintrc.json 或您使用的任何相关配置中，只需指定：

"require-atomic-updates": "off"

Answer 3

我不认为这是一个错误。假设您的代码 sn-p 包含在异步函数中，即doRequest。

只要在doRequest 之外定义了ctx 变量，ctx.body 赋值就处于竞争状态。因为无法保证分配给ctx.body 的最后一个值属于doRequest 的最后一次调用。

我写了a blog post 关于这个竞争条件。有两种方法可以避免此警告

方法一：将ctx移动到doRequest体中，然后在函数末尾返回ctx值。

方法二：使用promise-base-semaphore pattern

let ctxPromise

每当您发出请求时，请致电ctxPromise = doRequest()。