Express.js多并发请求下数据跨请求泄漏问题排查求助

核心原因 #

问题出在builder.js的全局状态共享上：如果你的buildFile函数里用了全局变量存储API返回数据、请求参数或临时内容，并发请求时，后续请求的异步操作会覆盖全局变量里的前序请求数据。当仅调用1次API时，异步执行窗口短，覆盖概率低；调用2次及以上时，异步等待时间变长，多个请求的异步步骤交叉执行，数据覆盖的问题就会彻底暴露。

修复步骤 #

1. 移除builder.js中的全局变量 #

把所有和请求相关的变量移到buildFile函数的局部作用域，确保每个请求的状态完全独立：

错误写法（存在全局变量）：

// builder.js 错误示例
let apiResp1 = {};
let apiResp2 = {};

async function buildFile(reqParams) {
  apiResp1 = await fetchThirdPartyAPI1(reqParams);
  apiResp2 = await fetchThirdPartyAPI2(reqParams);
  return assembleFile(apiResp1, apiResp2);
}

修复后写法：

// builder.js 正确示例
async function buildFile(reqParams) {
  // 所有数据都存于函数局部作用域，每个请求独立拥有
  const apiResp1 = await fetchThirdPartyAPI1(reqParams);
  const apiResp2 = await fetchThirdPartyAPI2(reqParams);
  return assembleFile(apiResp1, apiResp2);
}

2. 确保Express路由的请求隔离 #

在/file接口中，每个请求都触发独立的buildFile调用，不共享任何处理对象或变量：

// server.js
const express = require('express');
const builder = require('./builder');
const app = express();

app.get('/file', async (req, res) => {
  try {
    // 提取当前请求的唯一参数，确保每个请求的处理上下文独立
    const uniqueParams = getRequestParams(req);
    const fileContent = await builder.buildFile(uniqueParams);
    res.send(fileContent);
  } catch (err) {
    res.status(500).send('Failed to generate file');
  }
});

app.listen(3000);

3. 并发场景验证 #

用并行请求工具测试修复效果，比如：

# 并行发起5个带唯一参数的请求
seq 1 5 | xargs -I {} -P 5 wget http://localhost:3000/file?id={}

检查每个请求返回的文件内容是否匹配对应参数，确认无数据泄漏。

额外注意事项 #

• 若使用缓存，缓存键必须是请求的唯一标识（比如参数哈希、用户ID），避免不同请求命中同一缓存项。
• 涉及临时文件时，用UUID生成唯一文件名，防止并发请求覆盖文件内容。

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