Nodejs表达app,传播错误

时间:2013-07-04 15:23:45

标签: node.js express

我很想知道是否有将内部服务器错误传播回用户的最佳做法。 I.E.不是实际的错误(我记录了服务器端),但至少是500和消息。

我目前有:

  app.use(function(err, req, res, next) {
    console.error(err.stack);
    res.send(500, 'Internal server error, please contact server administrator.');
  });

除了如果抛出错误我的服务器崩溃(预期)之外,哪个很棒。

所以我目前正在将任何和所有错误传播回快速路线,即

  app.get('/api/users', function (req, res, next) {
    User.getUsers(function (err, users) { // err comes back in function
      // dont throw err here in callback
      // throw err;

      // Instead call next with the error to let express now something bad happened
      if (err) return next(err);

      res.json(users);
    });
  });

除非我有30条具有相同代码行的路由,否则这很有效。

  if (err) return next(err);

并不是一件大事,但让我想知道是否有一种“更好”的方式。

感谢。

1 个答案:

答案 0 :(得分:1)

编辑:OP希望避免将错误传播回快速路线。

从node.js 0.6.x开始,您可以使用domains,其行为类似于本地process.on('uncaughtException,...)`。已经有几个问题涉及这个问题,例如Unable to handle exception with node.js domains using express

诀窍是为每个请求附加一个单独的域:

app.use(function(req, res, next) {

  var reqDomain = domain.create();
  reqDomain.add(req);
  reqDomain.add(res);
  reqDomain.on('error', function(err) {
    console.log("Req Domain Error: " + err);
    res.send(500, err.message);
  });

  next();
});

每当发生异常时,都会调用处理的域错误,您仍然可以访问响应对象以发送5XX代码。

但是,正如文档所述,在发生这种情况时,应该在清理当前连接后立即终止node.js进程。

要考虑的另一件事是你需要确保你留在同一个域中。每当我打电话给外部模块时,我都会使用它:

exports.preserveDomain = function (fun) {
  var d = process.domain;

  return function () {
    var args = arguments;

    d.run(function () {
      fun.apply(null, args);
    });
  };
};