Add ability to configure PassportStrategy per Request

[Bnaya]

I'm submitting a...

[ ] Regression 
[ ] Bug report
[*] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

Unable to configure PassportStrategy per request

I want to be able to set custom PassportStrategy settings per request.

My use-case: The app is deployed on dynamic, multiple domains,
And i want to be able to set the callbackURL for oauth providers by the request.host, protocol, etc.

@Injectable({
  scope: Scope.REQUEST,
})
export class GoogleStrategy extends PassportStrategy(Strategy, 'google') {
  constructor(
    @Inject(REQUEST) private readonly request: import("express").Request,
    private readonly authService: AuthService,
  ) {
    super({
      clientID: 'CLIENT_ID', // <- Replace this with your client id
      clientSecret: 'CLIENT_SECRET', // <- Replace this with your client secret
      callbackURL: `${request.protocol}://${request.host}/auth/google/callback`,
      passReqToCallback: true,
      scope: ['profile'],
    });
  }

This code doesn't work, i get unknown strategy "google",
Because the strategies must be not Scope.REQUEST

[kamilmysliwiec]

EDIT

See this post: #95 (comment)

[cjancsar]

@kamilmysliwiec why are request scoped providers not compatible?

[spion]

The passport API is based on registering strategies to the (typically) global instance and passing the options when constructing the strategy

passport.use(new MyStrategy(options))

Therefore strategies are unfortunately not designed to have request-dependent options or to be instantiated per request.

You could recreate a new passport and re-register strategies for every single request, however, I don't think that's the intended use of the library.

[kamilmysliwiec]

Workaround: https://docs.nestjs.com/techniques/authentication#request-scoped-strategies