Differential D20030

Update Google OAuth adapter to move away from deprecated "Google+" APIs
ClosedPublic
Actions

Authored by epriestley on Jan 25 2019, 1:48 AM.

Tags

None

Referenced Files

	F15501910: D20030.id47836.diff
	Sun, Apr 13, 10:11 PM

	F15499943: D20030.id.diff
	Sun, Apr 13, 3:35 PM

	F15496466: D20030.id47863.diff
	Sun, Apr 13, 6:54 AM

	F15492521: D20030.diff
	Sat, Apr 12, 3:24 PM

	F15465348: D20030.id47836.diff
	Wed, Apr 2, 5:41 PM

	F15460787: D20030.id47836.diff
	Tue, Apr 1, 2:12 AM

	F15414783: D20030.diff
	Thu, Mar 20, 2:32 AM

	F15387591: D20030.id47863.diff
	Mar 15 2025, 2:39 AM

View All Files

Subscribers

dragoon20

reed

Details

Reviewers

amckinley
dragoon20

Maniphest Tasks

T13227: Figure out if Google Auth needs to be updated before Google+ shutdown on March 7, 2019

Commits

rPHU9144fc5a6896: (stable) Promote 2019 Week 4
rPHU9456d6683d33: Update Google OAuth adapter to move away from deprecated "Google+" APIs

Summary

Fixes T13227.

Test Plan

Screen Shot 2019-01-24 at 5.47.10 PM.png (807×1 px, 168 KB)

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 25 2019, 1:48 AM

Harbormaster completed remote builds in B21690: Diff 47836.Jan 25 2019, 1:49 AM

epriestley requested review of this revision.Jan 25 2019, 1:49 AM

epriestley added inline comments.Jan 25 2019, 1:50 AM

src/auth/PhutilGoogleAuthAdapter.php
150	None of these instructions seem to make sense anymore and I suspect this "Enable Google+" error is now impossible. Fingers crossed until evidence surfaces to suggest otherwise.

epriestley mentioned this in D20015: Adjust Google Auth Adapter to use Open ID Connect.Jan 25 2019, 1:51 AM

I tested this and it works marvelous, thanks, waiting for the patch.

Macro lgtmacro:

This revision is now accepted and ready to land.Jan 25 2019, 3:04 AM

Closed by commit rPHU9456d6683d33: Update Google OAuth adapter to move away from deprecated "Google+" APIs (authored by epriestley). · Explain WhyJan 28 2019, 5:40 PM

This revision was automatically updated to reflect the committed changes.

reed added a subscriber: reed.Feb 22 2019, 4:40 PM

Revision Contents
Changeset List

Path

Size

src/

auth/

PhutilGoogleAuthAdapter.php

92 lines

Diff 47863

View Options

src/auth/PhutilGoogleAuthAdapter.php

Update Google OAuth adapter to move away from deprecated "Google+" APIsClosedPublicActions