Node.js Express触发ERR_HTTP_HEADERS_SENT错误，刷新页面即断连求排查

核心报错触发原因 #

• 直接触发ERR_HTTP_HEADERS_SENT的位置是landing_page控制器：
  • 函数内首先同步执行了一次res.render('planner', [硬编码假数据])，此时Express已经完成响应头发送、响应体输出流程。
  • 后续db.getAllGoals()返回的Promise被resolve时，又第二次调用res.render('planner', [数据库返回数据])，此时再次尝试写入响应头就会抛出该错误。
• 该错误属于未捕获的进程级异常，会直接导致Node.js服务崩溃，因此刷新页面、切换路由时会立刻出现连接断开的问题。

其他逻辑问题 #

• landing_page内无意义调用db.removeGoal(req.body.user)：页面路由默认走GET请求，req.body为undefined，该调用会抛出参数错误，且删除操作与页面渲染逻辑完全无关。
• post_new_goal控制器调用db.addGoal后未返回任何响应，会导致POST请求一直挂起直到超时。

修复代码示例 #

修正后的landing_page控制器 #

exports.landing_page = function(req, res) {
    // 移除提前执行的硬编码渲染逻辑，仅在异步获取数据完成后返回一次响应
    db.getAllGoals().then((list) => {
        res.render('planner', {
            'title': 'Goal Planner',
            'planner': list
        });
        console.log('promise resolved');
    }).catch((err) => {
        console.log('promise rejected', err);
        // 异常场景下也要返回统一响应，避免请求挂起
        res.status(500).send('Failed to load planner data');
    }) 
    // 移除无意义的db.removeGoal调用
}

修正后的post_new_goal控制器 #

exports.post_new_goal = function(req, res) {
    console.log('processing post-new_entry controller');
    if (!req.body.typeOfExercise) {
        return res.status(400).send("Entries must have a type of exercise.");
    }
    // 数据写入完成后必须返回响应，避免请求挂起
    db.addGoal(req.body.user, req.body.typeOfExercise, req.body.goalDate, req.body.exerciseDuration)
        .then(() => res.redirect('/planner'))
        .catch(err => res.status(500).send('Failed to save goal'));
}

通用注意事项 #

• Express 框架规则：每个HTTP请求仅能发送一次响应，所有在响应发送后试图修改响应头、发送响应体的操作都会触发该报错。
• 所有异步逻辑的响应处理，必须放到Promise的回调链路内，禁止在异步逻辑外提前调用res系列方法。

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