Homestyx hydra
Diffusion hydra 308c4f240754

Fix "AphrontRequest->getRequestURI()" for requests with "x[]=1" parameters in…

Description

Fix "AphrontRequest->getRequestURI()" for requests with "x[]=1" parameters in the URI

Summary:
Ref T13250. See PHI1069. This is a small fix for getRequestURI() currently not working if the request includes "x[]=..." PHP-flavored array parameters, beacause they're parsed into arrays by $_GET and setQueryParams(...) no longer accepts nonscalars.

Instead, just parse the raw request URI.

Test Plan: Visited /search/hovercard/?phids[]=X, no more fatal. Dumped the resulting URI, saw it had the right value. Tried ?phids[]=x&x=1&x=1&x=1, saw the parameters correctly preserved.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13250

Differential Revision: https://secure.phabricator.com/D20147

Details

Provenance
epriestleyAuthored on Feb 12 2019, 8:31 AM
sirocylPushed on Oct 16 2024, 5:49 AM
Parents
R1:1fd69f788cee: Replace "getQueryParams()" callsites in Phabricator
Branches
Unknown
Tags
Unknown

Event Timeline