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

Make authorizationParams usable #4

Closed NicoleRauch closed 9 years ago

NicoleRauch commented 9 years ago

In the strategy, the variable "params" was redeclared after it was retrieved from the authorizationParams() method. This kept users of the strategy from adding non-standard parameters to the authentication request. Removing the redeclaration of the variable made the method authorizationParams() usable.

NicoleRauch commented 9 years ago

Meanwhile, I have cloned the project and published it as a separate node module. For details see https://github.com/NicoleRauch/openidconnect-for-passport