egg-cnode
egg-cnode copied to clipboard
cnodejs
passport 迁移
疑问
- egg-passport 登录时如何传递成功/失败消息有点疑问
- 返回 null 可跳转登录页，但目前消息传递有问题，可选实现为直接写 session
- 若抛出异常（401），是否考虑使用通用的错误页面
- 登录成功的页面需要认证后的数据，是否考虑通过中间件将 session 中的信息添加到 ctx.locals，减少路由中重复书写？
参考
- egg-passport 成功/失败信息
done(null, user, message)
// 此处不抛出异常则缺少 message, egg-passport 实现为 done(null, user)
handler 抛异常会通过 done(err) 返回 https://github.com/eggjs/egg-passport/blob/0afce5b0d5fbc107730e10dad6aff915c03cf079/lib/passport.js#L63
当然，你可以在 handler 里面实现 try catch 自行决定如何封装错误处理。
是否自定义 401 页面逻辑，完全由应用自身决定。
user 为 null 肯定就是登录失败了。
登录成功的页面需要认证后的数据，是否考虑通过中间件将 session 中的信息添加到 ctx.locals，减少路由中重复书写？
passport 不应该管数据保存到那里，这些都是后续应用去接管的操作。
@fengmk2
上面说的是两部分：
-
passport 通过以后获取的信息，可以考虑通过一个 egg 中间节来添加，渲染的时候不用重复在
ctx.render(template, locals)中添加，类似router.get(path, middleware, controller.method) -
passport-local 是 passport 的 handler 部分，这里如果不通过 done 方法传递 message，~~可以考虑直接写 message 到 session，是 passport 的 handler 处理了数据，和外部处理差不多~~，根据 @fengmk2 的建议，可以考虑通过封装 user 对象返回用户信息以及消息（验证成功/失败）并存储在 ctx.session.passport 中，此后可使用中间件可读取数据并消除 connect-flash 中使用ctx.session.message 带来的冲突
@fengmk2 使用 egg-passport-local 插件，試過直接拋異常如下：
// app.js
app.passport.verify(async (ctx, user) => {
throw new Error('找不到 user');
}
在 npm run dev 執行的環境下，網頁就回應 500 狀態碼。
@OnedayLiu 我正好在研究當 verify 失敗時要如何處理錯誤訊息讓前端可以顯示
https://github.com/eggjs/egg-passport/blob/0afce5b0d5fbc107730e10dad6aff915c03cf079/lib/passport.js#L63-L76
doVerify(req, user, done) {
const hooks = this._verifyHooks;
if (hooks.length === 0) return done(null, user);
(async () => {
const ctx = req.ctx;
for (const handler of hooks) {
user = await handler(ctx, user);
if (!user) {
break;
}
}
done(null, user);
})().catch(done);
}
可以先考慮在上列的 doVerify 範圍內增加捕捉 exception 然後改跑 done(err, user) 這樣就可以用 throw new Error() 丟錯誤訊息了。
可惜目前 doVerify 的程式碼段落還不是我完全看得懂的語法，所以無法直接動手改程式送 PR 了。
我試過改成這樣但還是不行，還是 500 錯誤跑到 node_modules/koa-onerror/index.js 出現 non-error thrown 錯誤訊息，看來我的認知還不夠完整
doVerify(req, user, done) {
const hooks = this._verifyHooks;
if (hooks.length === 0) return done(null, user);
(async () => {
const ctx = req.ctx;
for (const handler of hooks) {
user = await handler(ctx, user);
if (!user) {
break;
}
}
done(null, user);
}
})().catch(err => {
done(err.message, user);
});
}