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

Return all google profile data for the user #635

Closed
wants to merge 1 commit into from

Conversation

jperl
Copy link
Contributor

@jperl jperl commented Jan 25, 2013

It is useful inside In Accounts.onCreateUser to get the picture, if the email is verified, etc..

https://developers.google.com/accounts/docs/OAuth2Login#userinfocall

@hijonathan
Copy link

👍

@gschmidt
Copy link
Contributor

gschmidt commented Feb 4, 2013

Thanks @jperl! @avital and I talked about this and as discussed at #636, we think it'd be better to whitelist the fields. It would definitely be good to add the additional fields from Google though. Could you submit a pull request that specifically copies over the fields from Google that you think would be useful?

There's another question here, which is how these fields are synchronized/updated from Google after initial account creation. If you think that saving the values at account creation time is a good starting point, though, happy to let that slide for now (since people are already doing this manually.)

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

Successfully merging this pull request may close these issues.

None yet

3 participants