Node.js中Mongoose then链重定向报错：响应头重复修改问题排查

我编写了两段用Mongoose处理用户注册请求的Node.js代码，第一段运行时触发**“尝试在发送重定向后修改响应头”**的错误；修改后的第二段可正常执行重定向。我推测问题源于重复调用redirect()，但想明确第一段代码失效的具体原因。

第一段报错代码 #

exports.postSignup = (req, res) => {
  const email = req.body.email;
  const password = req.body.password;
  const confirmPassword = req.body.confirmPassword;
  // 检查用户是否已存在
  User.findOne({ emailAddress: email })
    .then((user) => {
      if (user) {
        console.log("用户已存在，请登录！");
        return res.redirect("/auth/login");
      }
      const newUser = new User({
        emailAddress: email,
        password: password,
        cart: {
          products: [],
        },
      });
      return newUser.save();
    })
    .then((result) => {
        console.log("用户创建成功");
        res.redirect("/auth/login");
    })
    .catch((e) => {
      console.log(e);
      throw e;
    });
};

修复后正常运行的代码 #

exports.postSignup = (req, res) => {
  const email = req.body.email;
  const password = req.body.password;
  const confirmPassword = req.body.confirmPassword;
  // 检查用户是否已存在
  User.findOne({ emailAddress: email })
    .then((user) => {
      if (user) {
        console.log("用户已存在，请登录！");
        return res.redirect("/auth/login");
      }
      const newUser = new User({
        emailAddress: email,
        password: password,
        cart: {
          products: [],
        },
      });
      return newUser.save();
    })
    .then((result) => {
      // 仅当用户创建成功时执行重定向
      if (result) {
        console.log("用户创建成功");
        res.redirect("/auth/login");
      }
    })
    .catch((e) => {
      console.log(e);
      throw e;
    });
};

问题根源与修复说明 #

第一段代码报错的核心原因 #

当查询到用户已存在时，第一个then回调里执行return res.redirect("/auth/login")，此时res.redirect()会立刻向客户端发送302重定向响应，HTTP响应流程已经完成。

但res.redirect()返回的是res对象（非Promise值），Promise链不会因此中断，会继续执行下一个then回调。这个回调里又调用了res.redirect("/auth/login")，试图再次修改响应头并发送重定向——而HTTP协议要求一个请求只能对应一个响应，此时响应已经发送完毕，再次操作响应对象就会触发错误。

修复代码的逻辑作用 #

第二段代码在第二个then回调中增加了if (result)判断：

• 当用户已存在时，第一个then返回的是res对象，第二个then的result就是这个res对象，但此时已经发送过一次重定向响应，if判断里的重定向逻辑不会执行，避免了重复操作响应对象。
• 当用户不存在时，第一个then返回的是newUser.save()的Promise，成功后result是保存后的用户文档，此时才会执行重定向，保证一个请求只发送一次响应。

更严谨的优化思路 #

除了通过result判断，还可以在第一个then里用else包裹创建用户的逻辑，彻底中断后续不必要的Promise链执行：

.then((user) => {
  if (user) {
    console.log("用户已存在，请登录！");
    return res.redirect("/auth/login");
  } else {
    const newUser = new User({
      emailAddress: email,
      password: password,
      cart: { products: [] },
    });
    return newUser.save();
  }
})

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