Saving context in application is an anti-pattern

[geekdada]

I found on some occasions, the Model being instantiated with the last request context, then I found this:

egg-oauth2-server/lib/server.js

Line 86 in 2d10b76

this.ctx = ctx;

You should never save ctx in application ever. This is my fix:

{
  getServer(ctx) {
    if (!ctx[SERVER]) {
      const { config, model: Model } = this;
      const model = new Model(ctx);
      ctx[SERVER] = new AuthServer(Object.assign(config, { model }));
      return ctx[SERVER];
    }
    return ctx[SERVER];
  }
}

[geekdada]

Or probably, use app/extend/context.js to get AuthServer attached on ctx.

[Azard]

ctx[SERVER] = new AuthServer(Object.assign(config, { model })); could create AuthServer every time, it's not performance.

[geekdada]

I'm a little confused, then what's this for?

egg-oauth2-server/lib/server.js

Line 77 in 2d10b76

const model = new Model(ctx);

If creating a new instance every time when a request comes is a bad approach, maybe you shouldn't have made Model has ctx as parameters in the first place.

[geekdada]

I found this PR oauthjs/node-oauth2-server#462

[rise0chen]

Has the problem been solved?