嵌套Promise错误捕获困惑：Node.js登录函数重构求助

核心问题1：getUserAccountByEmail 错误被吞掉 #

你在getUserAccountByEmail里手动封装了Promise，但内部User.findAll的catch只做了日志输出，没有调用reject把错误抛出去。这导致数据库查询出错时，这个Promise永远不会进入rejected状态，外层的catch自然捕获不到错误。

核心问题2：userLogin 错误无法冒泡 #

userLogin是async函数，但你手动套了一层new Promise，而且内部用嵌套的then结构，没有返回后续的Promise链。这种嵌套写法会让内层Promise的错误无法向上冒泡到外层的catch，必须每个then单独加catch才能捕获。

修复步骤 #

1. 修复getUserAccountByEmail

直接返回User.findAll的Promise，不用额外封装，同时确保所有错误都能被抛出：

function getUserAccountByEmail(email) {
  logger.info("Looking up user email");
  return User.findAll({
    where: { email },
    limit: 1,
    attributes: [
      "email", "password", "userId", 
      "stripeConnectAccountId", "isStripeAccountSet", "stripeCustomerId"
    ]
  }).then(userResults => {
    if (doesUserExist(userResults)) {
      return userResults;
    } else {
      console.log("user doesnt exist in getuseraccount");
      throw new Error("User Email Does Not Exist In Database");
    }
  }).catch(error => {
    console.log("Error Accessing Database: UserAccountByEmail");
    logger.error("Error in getUserAccountByEmail: " + error);
    // 重新抛出错误，让外层捕获
    throw error;
  });
}

2. 用async/await重构userLogin

async函数本身就返回Promise，完全不需要手动封装，用await替代嵌套then，错误可以统一向上冒泡：

async function userLogin(email, submittedPassword) {
  console.log("in user login");
  const userAccountResults = await getUserAccountByEmail(email);
  const { email: userEmail, password: storedPassword } = userAccountResults[0];
  
  await checkUserPassword(submittedPassword, storedPassword);
  
  const tokens = await generateJWTToken(userEmail);
  
  const userObject = await createUserDataObject(tokens, userAccountResults);
  
  return userObject;
}

3. 简化外层调用

因为userLogin已经是async函数，外层也可以用await简化，单个try/catch就能处理所有错误：

const loginUser = handleAsync(async (req, res, next) => {
  const { email: userEmail, password: submittedPassword } = req.body;
  try {
    const userObject = await userLogin(userEmail, submittedPassword);
    res.json(userObject);
  } catch (err) {
    res.status(401).send(err.message || err);
    console.log("any error inside userLogin: " + err);
  }
});

为什么这样能解决问题 #

• 去掉冗余的Promise封装，让Promise链自然传递，错误会自动冒泡到最近的catch。
• async/await让异步代码逻辑更清晰，错误处理更集中，单个try/catch就能捕获所有异步操作的错误。
• 确保所有错误都被抛出，不会在中途被吞掉。

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