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
188 stars 173 forks source link

All parameters of OpenIDConnectStrategy are not given in the README.md documentation #95

Open laparn opened 2 years ago

laparn commented 2 years ago

Thanks for this nice passport library. I just used it and was blocked during some time, because I was not understanding why I couldn't get the email and displayName from a lemonldap-ng server which was configured for sending them in the claims. Finally, in looking in the examples (that are linked to in the github README.md but not on the passport-openidconnect page), I saw that there was a scope argument for the openIDConnectStrategy. Then I added scope: ['email', 'profile'] in the object provided to OpenIdStrategy and voilà ! it worked. So I think that all arguments of the OpenIdStrategy should be listed in a paragraph in the README.md file and for the poor programmer. That said, now, it works, so thanks a lot !