护照验证卡在控制器中？

Question

我尝试将我的路由转发到控制器，但它似乎不适用于 Passport.js

router.get('/login', (req, res, next) => UserController.getLogin(req, res, next));
router.post('/login', (req, res, next) => UserController.postLogin(req, res, next));

现在，唯一不起作用的路线是使用 Passport 的路线。

static getLogin(req: Request, res: Response, next: NextFunction) {
...
}
static postLogin(req: Request, res: Response, next: NextFunction) {

        passport.authenticate('local', {
            successRedirect: '/success',
            failureRedirect: '/failed'
        });
        // res.send('hello from POST'); would work
}

我正在使用 TypeScript

Answer 1

Passport 是异步的。它通常用作传递回调的中间件。例如，文档有这个例子：

app.post('/login', passport.authenticate('local', { successRedirect: '/',
                                                failureRedirect: '/login' }));

这里要记住的是passport.authenticate 返回一个接受(req, res, next) 的函数。然后它作用于该数据并在完成后调用next。在您的代码中，您正在调用authenticate（返回一个函数），然后什么也不做。我有一些建议。

首先是通过简化事物来减少噪音。根据框架，您通常可以传递一堆函数来处理路由。在这种情况下，您只需要一个。

router.post('/login', passport.authenticate('local', {
    successRedirect: '/success',
    failureRedirect: '/failed'
}))

如果您想做的不仅仅是身份验证，您可以传递更多功能。

router.post('/login', 
    passport.authenticate('local', {
        successRedirect: '/success',
        failureRedirect: '/failed'
    }),
    UserController.doThing // accepts (req, res, next)
)

您会注意到我没有创建一个匿名函数来将相同的 3 个参数传递给控制器​​。这不是必需的。在大多数情况下，它们是相同的。