Paths

Table of Contentst

Differential D20206

Fix Facebook login on mobile violating CSP after form redirect
ClosedPublic
Actions

Authored by epriestley on Feb 23 2019, 12:58 AM.

Tags

None

Referenced Files

	F15536861: D20206.diff
	Thu, Apr 24, 11:07 AM

	F15482096: D20206.diff
	Wed, Apr 9, 2:54 AM

	F15470361: D20206.id48244.diff
	Fri, Apr 4, 9:40 PM

	F15443624: D20206.id48244.diff
	Thu, Mar 27, 5:18 AM

	F15442919: D20206.id48244.diff
	Thu, Mar 27, 1:16 AM

	F15439159: D20206.id48246.diff
	Mar 26 2025, 6:03 AM

	F15435543: D20206.id.diff
	Mar 25 2025, 8:49 AM

	F15434265: D20206.id48246.diff
	Mar 25 2025, 2:49 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T13254: "Login with Facebook" may fail CSP after redirect to "m.facebook.com"

Commits

rP701a9bc339b9: Fix Facebook login on mobile violating CSP after form redirect

Summary

Fixes T13254. See that task for details.

Test Plan

Used iOS Simulator to do a login locally, didn't get blocked. Verified CSP includes "m.facebook.com".

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 23 2019, 12:58 AM

Harbormaster completed remote builds in B22101: Diff 48244.Feb 23 2019, 1:00 AM

epriestley requested review of this revision.Feb 23 2019, 1:00 AM

amckinley added inline comments.Feb 23 2019, 2:11 AM

src/applications/auth/provider/PhabricatorFacebookAuthProvider.php
117–127	Is this just incidentally dead code you're removing?

amckinley accepted this revision.Feb 23 2019, 2:15 AM

amckinley added inline comments.

src/applications/auth/provider/PhabricatorFacebookAuthProvider.php
117–127	Yeah it looks dead to me and `grep`.

This revision is now accepted and ready to land.Feb 23 2019, 2:15 AM

Yeah, sorry, no callsites per grep. And probably won't work without changes in a T6703 world anyway, since there might plausibly be multiple Facebook auth providers.

Closed by commit rP701a9bc339b9: Fix Facebook login on mobile violating CSP after form redirect (authored by epriestley). · Explain WhyFeb 23 2019, 1:25 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

auth/

provider/

PhabricatorFacebookAuthProvider.php

19 lines

Diff 48246

src/applications/auth/provider/PhabricatorFacebookAuthProvider.php

Loading...