Produce correct query strings from PhutilUri regardless of ini settings
ClosedPublic
Actions

Authored by richardvanvelzen on Oct 14 2015, 5:23 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Commits

rPHUaee76238d694: Produce correct query strings from PhutilUri regardless of ini settings

Summary

http_build_query will use the separator set in arg_separator.output. When this is set to & (which happens in various environments) this causes query parameters to have the & doubly encoded.

Test Plan

-> php -r 'ini_set("arg_separator.output", "&amp;"); echo http_build_query([1, 2, 3], "", "&");'
0=1&1=2&2=3

Diff Detail

Repository

rPHU libphutil

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

richardvanvelzen updated this revision to Diff 34468.Oct 14 2015, 5:23 PM

richardvanvelzen retitled this revision from to Product correct query strings from PhutilUri regardless of ini settings.

richardvanvelzen updated this object.

richardvanvelzen edited the test plan for this revision. (Show Details)

richardvanvelzen added a reviewer: epriestley.

richardvanvelzen set the repository for this revision to rPHU libphutil.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptOct 14 2015, 5:23 PM

Herald added a subscriber: Korvin. · View Herald Transcript

richardvanvelzen retitled this revision from Product correct query strings from PhutilUri regardless of ini settings to Produce correct query strings from PhutilUri regardless of ini settings.Oct 14 2015, 5:24 PM

richardvanvelzen edited edge metadata.

oh PHP

This revision is now accepted and ready to land.Oct 14 2015, 5:25 PM

Closed by commit rPHUaee76238d694: Produce correct query strings from PhutilUri regardless of ini settings (authored by epriestley, committed by epriestley). · Explain WhyOct 14 2015, 5:26 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

parser/

PhutilURI.php

2 lines

Diff 34469

View Options

Produce correct query strings from PhutilUri regardless of ini settingsClosedPublicActions