Added sort order SORT_STRING for params in UriSigner

Tim van Densen · fabpot · commit 2040139db6c6 · 2016-01-07T14:43:59.000+01:00
diff --git a/src/Symfony/Component/HttpKernel/Tests/UriSignerTest.php b/src/Symfony/Component/HttpKernel/Tests/UriSignerTest.php
@@ -33,6 +33,7 @@ public function testCheck()
 
         $this->assertTrue($signer->check($signer->sign('http://example.com/foo')));
         $this->assertTrue($signer->check($signer->sign('http://example.com/foo?foo=bar')));
+        $this->assertTrue($signer->check($signer->sign('http://example.com/foo?foo=bar&0=integer')));
 
         $this->assertTrue($signer->sign('http://example.com/foo?foo=bar&bar=foo') === $signer->sign('http://example.com/foo?bar=foo&foo=bar'));
     }
diff --git a/src/Symfony/Component/HttpKernel/UriSigner.php b/src/Symfony/Component/HttpKernel/UriSigner.php
@@ -91,8 +91,8 @@ private function computeHash($uri)
 
     private function buildUrl(array $url, array $params = array())
     {
-        ksort($params);
-        $url['query'] = http_build_query($params);
+        ksort($params, SORT_STRING);
+        $url['query'] = http_build_query($params, '', '&');
 
         $scheme = isset($url['scheme']) ? $url['scheme'].'://' : '';
         $host = isset($url['host']) ? $url['host'] : '';

Original file line number	Diff line number	Diff line change
`@@ -33,6 +33,7 @@ public function testCheck()`
`33`	`33`
`34`	`34`	`$this->assertTrue($signer->check($signer->sign('http://example.com/foo')));`
`35`	`35`	`$this->assertTrue($signer->check($signer->sign('http://example.com/foo?foo=bar')));`
	`36`	`+ $this->assertTrue($signer->check($signer->sign('http://example.com/foo?foo=bar&0=integer')));`
`36`	`37`
`37`	`38`	`$this->assertTrue($signer->sign('http://example.com/foo?foo=bar&bar=foo') === $signer->sign('http://example.com/foo?bar=foo&foo=bar'));`
`38`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -91,8 +91,8 @@ private function computeHash($uri)`
`91`	`91`
`92`	`92`	`private function buildUrl(array $url, array $params = array())`
`93`	`93`	`{`
`94`		`- ksort($params);`
`95`		`- $url['query'] = http_build_query($params);`
	`94`	`+ ksort($params, SORT_STRING);`
	`95`	`+ $url['query'] = http_build_query($params, '', '&');`
`96`	`96`
`97`	`97`	`$scheme = isset($url['scheme']) ? $url['scheme'].'://' : '';`
`98`	`98`	`$host = isset($url['host']) ? $url['host'] : '';`