Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
laparn opened this issue Mar 16, 2022 · 0 comments
Labels
documentation Improvements or additions to documentation

Comments

@laparn
Copy link

laparn commented Mar 16, 2022

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 !

@jaredhanson jaredhanson added the documentation Improvements or additions to documentation label Mar 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants