Skip to content
This repository has been archived by the owner on Jun 7, 2024. It is now read-only.

Add google oauth2 authentication #569

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Add google oauth2 authentication #569

wants to merge 14 commits into from

Conversation

suntehnik
Copy link

This commit adds google oauth2 authentication method for usegrid core services. It was tested using REST client with my android app and works as expected.
User profile, created by authenticating with google has it's user first and last names, picked form google plus profile, as well as user avatar and e-mail.

suntehnik and others added 14 commits March 1, 2017 09:31
…ction size exceeds the limit. Add QueryAnalyzer tests.
…ided ( you are allowed to create new admin users with just an email and it will invoke a reset password flow).
…eries and email addresses. See: USERGRID-29 USERGRID-1041 USERGRID-1117 USERGRID-1338
…s the in memory impl causes duplicate messgae processing quite often at the moment.

 - includes making all the tests work without in-memory queue fronting the database queue which really means adding some more delay in tests
 - the tests now are actually faster now because the original refreshIndex() created and queried random entities which took longer in most cases
 - uncommented the checkReceipts function in Notification tests as these are now passing, with an added fix for duplicate receipt creation
 - some logging updates in the distributed database queueing impl (Qakka)
 - increased the default take to 500 from the queue when DISTRIBUTED database queueing is configured ( which is the default now )
 - Notifications Queue Listener thread names have a random identifier in included
 - reduced the DISTRIBUTED database queueing default long poll to 1 second from 5 seconds
@michaelarusso
Copy link
Member

Can you re-create another PR with just your commit included? It would be great to add Google as another authentication provider.

@suntehnik
Copy link
Author

Yeah, sure!

@suntehnik
Copy link
Author

Done at #574

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants