在Express中能否在内部Promise执行完成前提前返回响应给客户端

核心结论 #

可以在Promise执行完成前就返回响应，你的场景下这种写法不会造成内存泄漏，但存在一个必须修复的严重隐患。

具体说明 #

不存在内存泄漏的原因 #

你代码里的repository.save(user)是独立的异步任务，会被Node.js的事件循环托管执行，执行过程中仅持有user对象的引用，没有绑定请求的req/res上下文，任务执行完成（无论成功失败）后相关资源都会被垃圾回收器正常释放，不会出现内存泄漏问题。

必须修复的隐患 #

你当前的写法没有给异步任务加错误捕获，一旦repository.save执行报错，会抛出未捕获的Promise拒绝（unhandledRejection），Node.js v15及以上版本遇到这种情况会直接终止进程，导致整个服务挂掉。

你只需要给这个Promise加个catch回调（或者在回调里打印错误日志方便排查问题）即可，修改后的代码如下：

app.get('/', (req, res) => {
  const user = new User();
  user.firstName = "Timber";
  user.lastName = "Saw";
  user.age = 25;
  // 加catch捕获错误，避免未捕获的异常终止进程
  repository.save(user).catch(err => {
    // 这里可以按需打印错误日志，不影响返回响应
    console.error('用户写入失败', err)
  });
  res.send('Hello World!')
})

其他注意事项 #

• 如果你的服务在数据库写入操作还没执行完就重启，这部分未完成的写入数据会丢失，你已经明确可以接受写入失败的话该风险可以忽略。
• 如果接口QPS很高，短时间内堆积大量未完成的数据库写入请求，可能会占满数据库连接池，影响其他需要同步等待数据库返回的接口性能，这种情况可以引入本地内存队列或者独立的消息队列做削峰处理，属于高并发场景的优化项，普通流量场景无需担心。

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