Remove BC layer for HttpFoundation < 6.1

GromNaN · GromNaN · commit 746c3fdf29d7 · 2023-06-30T01:58:13.000+02:00
diff --git a/src/Symfony/Component/Security/Http/Authenticator/FormLoginAuthenticator.php b/src/Symfony/Component/Security/Http/Authenticator/FormLoginAuthenticator.php
@@ -74,7 +74,7 @@ public function supports(Request $request): bool
     {
         return ($this->options['post_only'] ? $request->isMethod('POST') : true)
             && $this->httpUtils->checkRequestPath($request, $this->options['check_path'])
-            && ($this->options['form_only'] ? 'form' === (method_exists(Request::class, 'getContentTypeFormat') ? $request->getContentTypeFormat() : $request->getContentType()) : true);
+            && ($this->options['form_only'] ? 'form' === $request->getContentTypeFormat() : true);
     }
 
     public function authenticate(Request $request): Passport
diff --git a/src/Symfony/Component/Security/Http/Authenticator/JsonLoginAuthenticator.php b/src/Symfony/Component/Security/Http/Authenticator/JsonLoginAuthenticator.php
@@ -66,7 +66,7 @@ public function supports(Request $request): ?bool
     {
         if (
             !str_contains($request->getRequestFormat() ?? '', 'json')
-            && !str_contains((method_exists(Request::class, 'getContentTypeFormat') ? $request->getContentTypeFormat() : $request->getContentType()) ?? '', 'json')
+            && !str_contains($request->getContentTypeFormat() ?? '', 'json')
         ) {
             return false;
         }
diff --git a/src/Symfony/Component/Security/Http/composer.json b/src/Symfony/Component/Security/Http/composer.json
@@ -19,7 +19,7 @@
         "php": ">=8.1",
         "symfony/deprecation-contracts": "^2.5|^3",
         "symfony/security-core": "^6.3|^7.0",
-        "symfony/http-foundation": "^5.4|^6.0|^7.0",
+        "symfony/http-foundation": "^6.2|^7.0",
         "symfony/http-kernel": "^6.3|^7.0",
         "symfony/polyfill-mbstring": "~1.0",
         "symfony/property-access": "^5.4|^6.0|^7.0"

Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ public function supports(Request $request): bool`
`74`	`74`	`{`
`75`	`75`	`return ($this->options['post_only'] ? $request->isMethod('POST') : true)`
`76`	`76`	`&& $this->httpUtils->checkRequestPath($request, $this->options['check_path'])`
`77`		`- && ($this->options['form_only'] ? 'form' === (method_exists(Request::class, 'getContentTypeFormat') ? $request->getContentTypeFormat() : $request->getContentType()) : true);`
	`77`	`+ && ($this->options['form_only'] ? 'form' === $request->getContentTypeFormat() : true);`
`78`	`78`	`}`
`79`	`79`
`80`	`80`	`public function authenticate(Request $request): Passport`
Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,7 @@ public function supports(Request $request): ?bool`
`66`	`66`	`{`
`67`	`67`	`if (`
`68`	`68`	`!str_contains($request->getRequestFormat() ?? '', 'json')`
`69`		`- && !str_contains((method_exists(Request::class, 'getContentTypeFormat') ? $request->getContentTypeFormat() : $request->getContentType()) ?? '', 'json')`
	`69`	`+ && !str_contains($request->getContentTypeFormat() ?? '', 'json')`
`70`	`70`	`) {`
`71`	`71`	`return false;`
`72`	`72`	`}`