-
Notifications
You must be signed in to change notification settings - Fork 114
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
Google? #14
Comments
Would love to do this, haven't got the time at the mo, but I could give someone pointers if they wanted to help work on a strategy contribution? |
I've wrote an authentication strategy for Google. I hope this is useful. https://github.com/masahiroh/connect-auth/commit/1e5472b76ab76e894f382bfe40f9644be190454e |
Awesome, I will check it out. |
Hiyah, thanks for this I've pulled the changes in, I think its a good stepping stone towards Google SSO support, but the real solution I think requires OpenID support (which there are at least 2 node modules being written to provide.) When this comes along the authentication flow with google will be a lot tidier..with the OAuth approach we will keep getting prompted, which is a bit yukky atm :( |
Indeed. I've added Google OAuth because I want to use Data API, but OpenID support is required for SSO. I don't know much about the OpenID's specification, but I'll give it a try:) |
Has anyone made any progress with this? There's what seems to be a fairly complete openid library here. |
I've not tried it myself, might be interesting now with G+ around. |
There are now at least 2 different google SSO strategies provided, and a separate OpenId strategy :) |
Will Google be added to the list anytime soon?
or has somebody got this working with Google authentication?
The text was updated successfully, but these errors were encountered: