Homec4science

Add Google as an OAuth2 provider (BETA)

Authored by epriestley <git@epriestley.com> on Sep 9 2011, 01:37.

Description

Add Google as an OAuth2 provider (BETA)

Summary:
This is pretty straightforward, except:

  • We need to request read/write access to the address book to get the account

ID (which we MUST have) and real name, email and account name (which we'd like
to have). This is way more access than we should need, but there's apparently no
"get_loggedin_user_basic_information" type of call in the Google API suite (or,
at least, I couldn't find one).

  • We can't get the profile picture or profile URI since there's no Plus API

access and Google users don't have meaningful public pages otherwise.

  • Google doesn't save the fact that you've authorized the app, so every time

you want to login you need to reaffirm that you want to give us silly amounts of
access. Phabricator sessions are pretty long-duration though so this shouldn't
be a major issue.

Test Plan:

  • Registered, logged out, and logged in with Google.
  • Registered, logged out, and logged in with Facebook / Github to make sure I

didn't break anything.

  • Linked / unlinked Google accounts.

Reviewers: Makinde, jungejason, nh, tuomaspelkonen, aran

Reviewed By: aran

CC: aran, epriestley, Makinde

Differential Revision: 916

Details

Committed
epriestley <git@epriestley.com>Sep 14 2011, 16:32
Pushed
aubortJan 31 2017, 17:16
Parents
rPH4da43b31a307: Add Mercurial repository configuration and local pull support
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH1620bce842ea: Add Google as an OAuth2 provider (BETA) (authored by epriestley <git@epriestley.com>).Sep 14 2011, 16:32