Firestore查询无匹配数据时登录控制陷入无限循环问题

看起来你的问题出在Firestore查询结果的处理逻辑上，我帮你拆解下问题所在：

核心问题分析 #

当没有匹配的用户数据时，Firestore返回的snapshot是空的，你的forEach循环根本不会执行——这就导致代码既没有触发res.redirect('/main')，也没有执行else里的res.redirect('back')，请求一直处于挂起状态，前端得不到响应，自然会出现“无限循环/无法跳转”的现象。

另外还有个小误区：snapshot.forEach里的doc本身就是存在的文档（只有存在的文档才会出现在查询快照里），所以if (doc.exists)这个判断其实是多余的。

修复后的后端代码 #

// 先处理密码加密
const encryptedPw = sha256(req.body.pw);

// 查询用户数据
db.collection('users')
  .where('id', '==', req.body.id)
  .where('pw', '==', encryptedPw)
  .get()
  .then(snapshot => {
    // 先判断是否有匹配的文档
    if (!snapshot.empty) {
      // 因为账号应该是唯一的，直接取第一个匹配的文档
      const doc = snapshot.docs[0];
      const userData = doc.data();
      // 设置session
      req.session.user = {
        'id': userData.id,
        'name': userData.name,
        'auth': userData.auth,
        'is_guest': userData.is_guest
      };
      res.redirect('/main');
    } else {
      // 没有匹配数据，跳转回登录页
      res.redirect('back');
    }
  })
  .catch(err => {
    console.error('登录查询出错:', err);
    res.redirect('back');
  });

关键修复点 #

• 判断快照是否为空：用snapshot.empty直接检查是否有匹配结果，替代原来的forEach遍历判断，确保无匹配时能触发跳转逻辑
• 处理唯一账号场景：因为用户ID应该是唯一的，直接取snapshot.docs[0]即可，不需要遍历所有结果
• 确保所有分支都有响应：不管是匹配成功、无匹配还是查询出错，都明确调用res.redirect，避免请求挂起

额外建议 #

• 前端可以给表单添加简单的加载状态，避免用户多次点击提交
• 考虑给用户ID字段添加Firestore唯一索引，确保不会出现重复账号的情况

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