bug #27152 [HttpFoundation] use brace-style regex delimiters (xabbuh)

fabpot · fabpot · commit 423a638f3a82 · 2018-05-06T19:34:00.000+02:00
This PR was merged into the 2.7 branch. Discussion ---------- [HttpFoundation] use brace-style regex delimiters | Q | A | ------------- | --- | Branch? | 2.7 | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | #27107 | License | MIT | Doc PR | Commits ------- ae62d9b use brace-style regex delimiters
diff --git a/src/Symfony/Component/HttpFoundation/Request.php b/src/Symfony/Component/HttpFoundation/Request.php
@@ -581,7 +581,7 @@ public static function getTrustedProxies()
     public static function setTrustedHosts(array $hostPatterns)
     {
         self::$trustedHostPatterns = array_map(function ($hostPattern) {
-            return sprintf('#%s#i', $hostPattern);
+            return sprintf('{%s}i', $hostPattern);
         }, $hostPatterns);
         // we need to reset trusted hosts on trusted host patterns change
         self::$trustedHosts = array();
diff --git a/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php b/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php
@@ -18,6 +18,11 @@
 
 class RequestTest extends TestCase
 {
+    protected function tearDown()
+    {
+        Request::setTrustedHosts(array());
+    }
+
     public function testInitialize()
     {
         $request = new Request();
@@ -1871,9 +1876,15 @@ public function testTrustedHosts()
 
         $request->headers->set('host', 'subdomain.trusted.com');
         $this->assertEquals('subdomain.trusted.com', $request->getHost());
+    }
 
-        // reset request for following tests
-        Request::setTrustedHosts(array());
+    public function testSetTrustedHostsDoesNotBreakOnSpecialCharacters()
+    {
+        Request::setTrustedHosts(array('localhost(\.local){0,1}#,example.com', 'localhost'));
+
+        $request = Request::create('/');
+        $request->headers->set('host', 'localhost');
+        $this->assertSame('localhost', $request->getHost());
     }
 
     public function testFactory()

Original file line number	Diff line number	Diff line change
`@@ -581,7 +581,7 @@ public static function getTrustedProxies()`
`581`	`581`	`public static function setTrustedHosts(array $hostPatterns)`
`582`	`582`	`{`
`583`	`583`	`self::$trustedHostPatterns = array_map(function ($hostPattern) {`
`584`		`- return sprintf('#%s#i', $hostPattern);`
	`584`	`+ return sprintf('{%s}i', $hostPattern);`
`585`	`585`	`}, $hostPatterns);`
`586`	`586`	`// we need to reset trusted hosts on trusted host patterns change`
`587`	`587`	`self::$trustedHosts = array();`