fixed Via regex

thecaliskan · thecaliskan · commit d990eb9a7511 · 2025-05-26T11:39:29.000+03:00
diff --git a/src/Symfony/Component/HttpFoundation/Request.php b/src/Symfony/Component/HttpFoundation/Request.php
@@ -1384,7 +1384,7 @@ public function isMethodCacheable(): bool
     public function getProtocolVersion(): ?string
     {
         if ($this->isFromTrustedProxy()) {
-            preg_match('~^(HTTP/)?([1-9]\.[0-9]) ~', $this->headers->get('Via') ?? '', $matches);
+            preg_match('~^(HTTP/)?([1-9]\.[0-9])\b~', $this->headers->get('Via') ?? '', $matches);
 
             if ($matches) {
                 return 'HTTP/'.$matches[2];
diff --git a/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php b/src/Symfony/Component/HttpFoundation/Tests/RequestTest.php
@@ -2417,6 +2417,8 @@ public static function protocolVersionProvider()
             'trusted with via and protocol name' => ['HTTP/2.0', true, 'HTTP/1.0 fred, HTTP/1.1 nowhere.com (Apache/1.1)', 'HTTP/1.0'],
             'trusted with broken via' => ['HTTP/2.0', true, 'HTTP/1^0 foo', 'HTTP/2.0'],
             'trusted with partially-broken via' => ['HTTP/2.0', true, '1.0 fred, foo', 'HTTP/1.0'],
+            'trusted with simple via' => ['HTTP/2.0', true, 'HTTP/1.0', 'HTTP/1.0'],
+            'trusted with only version via' => ['HTTP/2.0', true, '1.0', 'HTTP/1.0'],
         ];
     }
 

Original file line number	Diff line number	Diff line change
`@@ -1384,7 +1384,7 @@ public function isMethodCacheable(): bool`
`1384`	`1384`	`public function getProtocolVersion(): ?string`
`1385`	`1385`	`{`
`1386`	`1386`	`if ($this->isFromTrustedProxy()) {`
`1387`		`- preg_match('~^(HTTP/)?([1-9]\.[0-9]) ~', $this->headers->get('Via') ?? '', $matches);`
	`1387`	`+ preg_match('~^(HTTP/)?([1-9]\.[0-9])\b~', $this->headers->get('Via') ?? '', $matches);`
`1388`	`1388`
`1389`	`1389`	`if ($matches) {`
`1390`	`1390`	`return 'HTTP/'.$matches[2];`
Original file line number	Diff line number	Diff line change
`@@ -2417,6 +2417,8 @@ public static function protocolVersionProvider()`
`2417`	`2417`	`'trusted with via and protocol name' => ['HTTP/2.0', true, 'HTTP/1.0 fred, HTTP/1.1 nowhere.com (Apache/1.1)', 'HTTP/1.0'],`
`2418`	`2418`	`'trusted with broken via' => ['HTTP/2.0', true, 'HTTP/1^0 foo', 'HTTP/2.0'],`
`2419`	`2419`	`'trusted with partially-broken via' => ['HTTP/2.0', true, '1.0 fred, foo', 'HTTP/1.0'],`
	`2420`	`+ 'trusted with simple via' => ['HTTP/2.0', true, 'HTTP/1.0', 'HTTP/1.0'],`
	`2421`	`+ 'trusted with only version via' => ['HTTP/2.0', true, '1.0', 'HTTP/1.0'],`
`2420`	`2422`	`];`
`2421`	`2423`	`}`
`2422`	`2424`