relax auth bearer format requirements

xabbuh · xabbuh · commit 8b0b581007bb · 2020-10-20T15:29:34.000+02:00
diff --git a/src/Symfony/Component/HttpClient/HttpClientTrait.php b/src/Symfony/Component/HttpClient/HttpClientTrait.php
@@ -110,8 +110,8 @@ private static function prepareRequest(?string $method, ?string $url, array $opt
             throw new InvalidArgumentException(sprintf('Option "auth_basic" must be string or an array, "%s" given.', \gettype($options['auth_basic'])));
         }
 
-        if (isset($options['auth_bearer']) && (!\is_string($options['auth_bearer']) || !preg_match('{^[-._=:~+/0-9a-zA-Z]++$}', $options['auth_bearer']))) {
-            throw new InvalidArgumentException(sprintf('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, '.(\is_string($options['auth_bearer']) ? 'invalid string given.' : '"%s" given.'), \gettype($options['auth_bearer'])));
+        if (isset($options['auth_bearer']) && (!\is_string($options['auth_bearer']))) {
+            throw new InvalidArgumentException(sprintf('Option "auth_bearer" must be a string , "%s" given.', \gettype($options['auth_bearer'])));
         }
 
         if (isset($options['auth_basic'], $options['auth_bearer'])) {
diff --git a/src/Symfony/Component/HttpClient/Tests/HttpClientTraitTest.php b/src/Symfony/Component/HttpClient/Tests/HttpClientTraitTest.php
@@ -179,17 +179,10 @@ public function testAuthBearerOption()
     public function testInvalidAuthBearerOption()
     {
         $this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');
-        $this->expectExceptionMessage('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, "object" given.');
+        $this->expectExceptionMessage('Option "auth_bearer" must be a string , "object" given.');
         self::prepareRequest('POST', 'http://example.com', ['auth_bearer' => new \stdClass()], HttpClientInterface::OPTIONS_DEFAULTS);
     }
 
-    public function testInvalidAuthBearerValue()
-    {
-        $this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');
-        $this->expectExceptionMessage('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, invalid string given.');
-        self::prepareRequest('POST', 'http://example.com', ['auth_bearer' => "a\nb"], HttpClientInterface::OPTIONS_DEFAULTS);
-    }
-
     public function testSetAuthBasicAndBearerOptions()
     {
         $this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');

Original file line number	Diff line number	Diff line change
`@@ -110,8 +110,8 @@ private static function prepareRequest(?string $method, ?string $url, array $opt`
`110`	`110`	`throw new InvalidArgumentException(sprintf('Option "auth_basic" must be string or an array, "%s" given.', \gettype($options['auth_basic'])));`
`111`	`111`	`}`
`112`	`112`
`113`		`- if (isset($options['auth_bearer']) && (!\is_string($options['auth_bearer']) \|\| !preg_match('{^[-._=:~+/0-9a-zA-Z]++$}', $options['auth_bearer']))) {`
`114`		`- throw new InvalidArgumentException(sprintf('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, '.(\is_string($options['auth_bearer']) ? 'invalid string given.' : '"%s" given.'), \gettype($options['auth_bearer'])));`
	`113`	`+ if (isset($options['auth_bearer']) && (!\is_string($options['auth_bearer']))) {`
	`114`	`+ throw new InvalidArgumentException(sprintf('Option "auth_bearer" must be a string , "%s" given.', \gettype($options['auth_bearer'])));`
`115`	`115`	`}`
`116`	`116`
`117`	`117`	`if (isset($options['auth_basic'], $options['auth_bearer'])) {`
Original file line number	Diff line number	Diff line change
`@@ -179,17 +179,10 @@ public function testAuthBearerOption()`
`179`	`179`	`public function testInvalidAuthBearerOption()`
`180`	`180`	`{`
`181`	`181`	`$this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');`
`182`		`- $this->expectExceptionMessage('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, "object" given.');`
	`182`	`+ $this->expectExceptionMessage('Option "auth_bearer" must be a string , "object" given.');`
`183`	`183`	`self::prepareRequest('POST', 'http://example.com', ['auth_bearer' => new \stdClass()], HttpClientInterface::OPTIONS_DEFAULTS);`
`184`	`184`	`}`
`185`	`185`
`186`		`- public function testInvalidAuthBearerValue()`
`187`		`- {`
`188`		`- $this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');`
`189`		`- $this->expectExceptionMessage('Option "auth_bearer" must be a string containing only characters from the base 64 alphabet, invalid string given.');`
`190`		`- self::prepareRequest('POST', 'http://example.com', ['auth_bearer' => "a\nb"], HttpClientInterface::OPTIONS_DEFAULTS);`
`191`		`- }`
`192`		`-`
`193`	`186`	`public function testSetAuthBasicAndBearerOptions()`
`194`	`187`	`{`
`195`	`188`	`$this->expectException('Symfony\Component\HttpClient\Exception\InvalidArgumentException');`