-
Notifications
You must be signed in to change notification settings - Fork 20
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
Force login via ?prompt=login query string #16
Comments
In order to enable forced login prompting parameter, e.g.: &prompt=login, see Issue jaredhanson#16.
Okay a bit of learning on my side, and I've added this functionality in the above pull request, can be used to force login as follows: app.get('/auth/windowslive', |
Hey @martinlaukkanen can you link to where that value for |
It seems like I can't force the selection no matter what. I think authorizationParams needs to be implemented here. |
Hi, not sure where I originally found that for the above pull request, but a quick look at the MS documentation and they list a few parameters not detailed in the stackoverflow link above: That document refers to the new MS Auth endpoints and not necessarily the old Live/Passport endpoints, but that option works for both in my tests. |
Ah thanks @martinlaukkanen! |
Hi,
How can I inject options that will force login on each authentication attempt?
Manually you would just append: ?prompt=login to the url.
E.g.:
https://login.live.com/oauth20_authorize.srf?response_type=code&redirect_uri=[.....]&prompt=login
But passing that to the 'authorizationURL' option results in a malformed URL as the prompt query string param is malformed; e.g.: https://login.live.com/oauth20_authorize.srf?prompt=login?response_type=code&redirect_uri=[...]
I can see exactly what it's doing (string concatenation) but can't see where in the code?
Thanks,
The text was updated successfully, but these errors were encountered: