Page MenuHomePhabricator

Straighten out absolute/relative URIs in login providers
ClosedPublic

Authored by epriestley on Jan 23 2014, 7:22 PM.
Tags
None
Referenced Files
F18839245: D8045.id.diff
Oct 27 2025, 3:15 PM
F18834800: D8045.diff
Oct 26 2025, 10:52 AM
F18734565: D8045.id.diff
Sep 30 2025, 10:56 PM
F18623338: D8045.id.diff
Sep 15 2025, 3:24 PM
F18613412: D8045.diff
Sep 14 2025, 11:10 AM
F18594060: D8045.diff
Sep 12 2025, 4:52 PM
F18557137: D8045.id.diff
Sep 8 2025, 7:52 PM
F18513128: D8045.diff
Sep 5 2025, 9:12 AM
Subscribers

Details

Summary

Ref T4339. Login providers use absolute URIs, but the ones that rely on local form submits should not, because we want to include CSRF tokens where applicable.

Instead, make the default be relative URIs and turn them into absolute ones for the callback proivders.

Test Plan

Clicked, like, every login button.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Jan 23 2014, 7:23 PM
  • Fix a couple more cases.