feat: 增加 egg-passport-local example

[OnedayLiu]

Checklist

• npm test passes
• tests and/or benchmarks are included
• documentation is changed or added
• commit message follows commit guidelines

Affected core subsystem(s)

Description of change

增加 egg-passport-local example

[atian25]

user.id 这个取值不对吧

[OnedayLiu]

我的想法是为了跟其他 strategy 的展示结果对应

// home.js
Logined user: <img src="${ctx.user.photo}"> ${ctx.user.displayName} / ${ctx.user.id} | <a href="/logout">Logout</a>

[atian25]

user.id 的语义应该是用户唯一 ID，不应该是你这个 local 的取值吧

[OnedayLiu]

egg-passport-local 只有 username,password,provider，user.id 得由应用方产生

// app.js
module.exports = app => {
  app.passport.verify(async (ctx, user) => {
    const existsUser = await ctx.model.User.findOne({  id: user.id });
    if (existsUser) {
      return existsUser;
    }
    const newUser = await ctx.service.user.register(user); // 生成 id
    return newUser;
  });
};

[atian25]

是的，所以你这里是没必要写这个 ID 的。
即使要显示 strategy，那应该是 ctx.user.provider 才对

[atian25]

改为 egg-view-nunjucks ?

[OnedayLiu]

OK，我改下

[atian25]

想起来了，之前我 app.passport.mount('twitter'); 的想法是会去判断对应的 strategy 如果提供了 mount 方法就用它的，否则用默认的。不过这样先也行吧

[OnedayLiu]

你的想法是不是这样：
可以由各个 strategy 实现自己的 mount 方法，用户调用 mount 的时候优先调用 strategy 的 mount，没有的的话才调用 egg-passport 默认的 mount

[atian25]

嗯

[atian25]

英文或干掉，其实用 egg-view-nunjucks 后就不用关了。

[OnedayLiu]

OK

[atian25]

不是，我的意思是，为什么你要把 user.id 赋值为 user.provider ?

[OnedayLiu]

搞错

[atian25]

egg-passport-local 那边还没发布吧? 那个 PR 要接手搞完

[OnedayLiu]

好的

[atian25]

rebase + ci