jaredhanson / passport-openidconnect

OpenID Connect authentication strategy for Passport and Node.js.
https://www.passportjs.org/packages/passport-openidconnect/?utm_source=github&utm_medium=referral&utm_campaign=passport-openidconnect&utm_content=about
MIT License
190 stars 175 forks source link

Pass preconfigure options to authenticate in addition to constructor #7

Closed JiCiT closed 2 years ago

JiCiT commented 9 years ago

I had a need to be able to configure for multiple clients, but which client isn't known until the point that authenticate is called. These changes let you pass those options to authenticate just like you do to the constructor.

jaredhanson commented 2 years ago

This use case can be accomplished now by passing strategy instances to passport.authenticate(), as described here: https://medium.com/passportjs/authenticate-using-strategy-instances-49e58d96ec8c

This approach avoids the need for each protocol-specific strategy to implement the types of changes proposed here. Closing.