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
F13239051: D8045.id18197.diff
Tue, May 21, 11:43 PM
F13238869: D8045.id18212.diff
Tue, May 21, 10:13 PM
F13238809: D8045.id18212.diff
Tue, May 21, 9:24 PM
F13227980: D8045.diff
Mon, May 20, 6:01 AM
F13224718: D8045.id.diff
Sun, May 19, 10:23 AM
F13220408: D8045.diff
Sun, May 19, 1:25 AM
F13202232: D8045.diff
Tue, May 14, 10:25 PM
F13201528: D8045.id18212.diff
Tue, May 14, 12:44 PM
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.