javascript - 在 promise 链中调用 express 中间件 next()

Question

我正在尝试构建一个参数中间件来加载请求的对象并将其附加到 request object，这样我就不必一遍又一遍地编写相同的代码。我正在使用 Sequelize 作为 ORM 来访问基于 Promise (bluebird) 的 MySQL 数据库。代码如下:

router.param('id', function (req, res, next, id) {
  return models.Object.findById(id)
    .then((object) => {
      //Check if the requested object was found
      if (object) {
        //Append the request object to the request object
        //for further usage in upcoming route handlers
        req.object = object
        return next()
      } else {
        //Throw an error, so that it can be caught
        //by the catch block of the promise chain
        let err = new Error('Object not found.')
        err.status = 404
        throw err
      }
    })
    //Catch any error and forward it to the
    //next express error handler
    .catch((err) => { return next(err) })
})

如您所见，我正在检查请求的对象是否存在，如果存在，我将其追加到 express 的请求对象中。如果不是，我会抛出一个错误。当我运行这段代码时，我收到警告:警告:在/*path goes here*/处的处理程序中创建了 promise，但未从中返回。在这一点上，我真的不知道如何摆脱这个错误。

编辑:

我尝试了 Hosar 的解决方案，但不幸的是警告仍然出现。这是我正在使用的确切代码:

router.param('aId', function (req, res, next, aId) {
  models.Authorization.findById(aId)
    .then((authorization) => {
      if (authorization) {
        req.authorization = authorization
        next()
      } else {
        let err = new Error('Not Found')
        err.status = 404
        next(err)
      }
    })
    .catch((err) => { return next(err) })
})

错误发生在我想调用不带任何参数的 next() 的那一行。完整的警告是:( Node :5504)警告:在 C:\Users\dkaiser 的处理程序中创建了 promise
\repos\lead\lead_backend\routes\auth.js:10:16 但没有从中返回

Answer 1

问题是你要返回一个 promise 。避免第一次返回。
尝试:

router.param('id', function (req, res, next, id) {
  models.Object.findById(id)
    .then((object) => {
      //Check if the requested object was found
      if (object) {
        //Append the request object to the request object
        //for further usage in upcoming route handlers
        req.object = object
        next()
      } else {
        //Throw an error, so that it can be caught
        //by the catch block of the promise chain
        let err = new Error('Object not found.')
        err.status = 404
        next(err);
      }
    })
    //Catch any error and forward it to the
    //next express error handler
    .catch((err) => { next(err) })
})