Express路由控制器内联正常但导出后失效——PassportJS登录路由问题

嘿，我一眼就看出问题出在哪了——你写的导出代码里，逗号运算符在搞鬼！

先给你拆解下为啥失效：
你的失效代码是这样的：

exports.loginAuth = passport.authenticate('local'), (req, res) => { console.log(req.user); }

在JS里，逗号运算符的作用是依次执行多个表达式，然后返回最后一个表达式的值。所以这行代码实际上等价于：

// 先执行passport.authenticate('local')，但结果没被关联到路由
passport.authenticate('local');
// 仅把后面的处理函数赋值给loginAuth
exports.loginAuth = (req, res) => { console.log(req.user); }

这就导致你的路由里根本没加载Passport的认证中间件，自然会报Cannot POST /users/login——因为Express找不到能处理这个POST请求的完整中间件链！

正确的导出方式 #

你需要把Passport认证中间件和后续的处理函数组合成一个中间件数组，这样导出后路由才能正确加载所有需要的逻辑：

方案1：数组形式导出（推荐，和内联写法逻辑完全一致）

exports.loginAuth = [
  passport.authenticate('local'),
  (req, res) => { 
    console.log(req.user);
    // 重要：别忘了给客户端返回响应，否则请求会一直挂起
    res.status(200).json({ message: '登录成功', user: req.user });
  }
];

然后在路由文件里这样引用：

const { loginAuth } = require('./你的导出文件路径');
// Express直接支持数组形式的中间件
router.post('/users/login', loginAuth);

方案2：自定义回调形式（适合需要灵活控制认证结果的场景）

如果你想对认证失败、系统错误等情况做自定义处理，可以用Passport的回调模式：

exports.loginAuth = (req, res, next) => {
  passport.authenticate('local', (err, user, info) => {
    // 处理认证过程中的系统错误
    if (err) return next(err);
    // 处理用户不存在/密码错误的情况
    if (!user) return res.status(401).json({ message: '账号或密码错误' });
    // 手动完成登录流程
    req.login(user, (err) => {
      if (err) return next(err);
      console.log(req.user);
      res.status(200).json({ message: '登录成功', user: req.user });
    });
  })(req, res, next);
};

这种方式不需要数组，直接在路由里引用即可：

router.post('/users/login', loginAuth);

最后提醒 #

不管用哪种方式，都要记得在处理函数里给客户端发送响应（比如res.json()或res.send()），否则客户端会一直处于等待状态，这也是很多新手容易忽略的点哦！

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