Page MenuHomePhabricator

Use "QUERY_STRING", not "REQUEST_URI", to parse raw request parameters
ClosedPublic

Authored by epriestley on Feb 28 2019, 6:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 29, 2:49 AM
Unknown Object (File)
Sun, Jun 29, 12:07 AM
Unknown Object (File)
Sat, Jun 28, 2:45 PM
Unknown Object (File)
May 8 2025, 2:48 AM
Unknown Object (File)
May 6 2025, 10:12 PM
Unknown Object (File)
Apr 24 2025, 8:23 AM
Unknown Object (File)
Apr 23 2025, 12:50 AM
Unknown Object (File)
Apr 20 2025, 10:50 AM
Subscribers
None

Details

Summary

Fixes T13260. "QUERY_STRING" and "REQUEST_URI" are similar for our purposes here, but our nginx documentation tells you to pass "QUERY_STRING" and doesn't tell you to pass "REQUEST_URI". We also use "QUERY_STRING" in a couple of other places already, and already have a setup check for it.

Use "QUERY_STRING" instead of "REQUEST_URI".

Test Plan

Visited /oauth/google/?a=b, got redirected with parameters preserved.

Diff Detail

Repository
rP Phabricator
Branch
query1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22149
Build 30267: Run Core Tests
Build 30266: arc lint + arc unit