Express.js中/test路由返回响应后全局错误处理仍触发404问题

在Express.js中编写了/test路由，该路由已通过res.send('hello')返回响应，按理应结束请求响应周期，但执行流程却进入了全局错误处理中间件，且err值为404，控制台打印出404但响应未抛出错误。相关代码及输出如下：

app.get('/test', (req, res, next) => {
  console.log('mw 1');
  res.send('hello');
});

// if none of the above routes matched
app.all('*', (req, res, next) => {
  next(404, `${req.originalUrl} not found`);
});

// Global error Handling
app.use((err, req, res, next) => {
  console.log('global error handler: ', err);
  res.status(500).json({
    status: '❌ error',
    message: err.message || 'Internal Server Error',
  });
});

输出：

mw 1
global error handler:  404

出现这个问题的核心原因有两点：

1. app.all('*')路由被意外执行

   • 若你使用的是Express 3.x版本，res.send方法会自动调用next()传递控制权，导致后续的app.all('*')路由因路径匹配（*匹配所有请求路径）而被执行。
   • 另一种可能是你在/test路由的回调中无意调用了next()，或误将app.all('*')写成了app.use('*')中间件——app.use定义的中间件会匹配所有路径前缀，即便前面的路由已经处理了请求，也会继续执行后续的app.use中间件。

2. 错误传递格式不符合Express规范

   • Express的错误处理中间件要求接收的err参数是标准Error对象，但你在app.all('*')中调用next(404, message)时，直接传递了数字404作为错误参数。虽然Express会识别这个值并触发错误处理中间件，但err并非标准Error对象，导致err.message为undefined，最终响应返回默认的"Internal Server Error"。
   • 同时，因为res.send('hello')已经发送了响应，错误处理中间件中res.status(500).json的操作会被忽略（HTTP响应只能发送一次），所以你看不到错误响应，但控制台会打印错误日志。

针对上述问题，可以通过以下步骤修复：

• 终止请求响应周期：在/test路由的回调中添加return，或调用next('route')跳过后续路由，避免app.all('*')被执行：

  app.get('/test', (req, res, next) => {
    console.log('mw 1');
    res.send('hello');
    return; // 终止当前回调，阻止后续路由执行
    // 或者使用 next('route'); 跳过后续路由（仅对路由生效）
  });
  

• 使用标准Error对象传递错误：修正app.all('*')中的错误传递逻辑，创建标准Error对象并附加状态码：

  app.all('*', (req, res, next) => {
    const err = new Error(`${req.originalUrl} not found`);
    err.status = 404;
    next(err);
  });
  

• 优化错误处理中间件：根据错误的状态码返回对应响应，避免统一返回500状态码：

  app.use((err, req, res, next) => {
    console.log('global error handler: ', err);
    const statusCode = err.status || 500;
    res.status(statusCode).json({
      status: '❌ error',
      message: err.message || 'Internal Server Error',
    });
  });
  

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