Merge branch '7.1' into 7.2

* 7.1:
  [Notifier] [Lox24] Fix request body format to JSON string
  inject the missing logger service
  fix handling of special "value" constraint option
  send the recipient phone number as an array
  [DependencyInjection] Fix ternary in AutowireCallable attribute
  [Security] Change to `BadCredentialsException` when empty username / password
  Add a note about the change in the default cache namespace generation to the upgrade guide
  fix test
  Fix CS
  [Messenger] Added postgres asset filter integration test
  change notifier type for brevo from chatter to texter
  avoid calling undefined built-in is_*() functions
  [AssetMapper] fix npm version constraint conversion
  Bump Symfony version to 7.1.2
  Update VERSION for 7.1.1
  Update CHANGELOG for 7.1.1

Author: derrabus

Authenticator/FormLoginAuthenticator.php

@@ -18,6 +18,7 @@
 use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
 use Symfony\Component\Security\Core\Authentication\Token\UsernamePasswordToken;
 use Symfony\Component\Security\Core\Exception\AuthenticationException;
+use Symfony\Component\Security\Core\Exception\BadCredentialsException;
 use Symfony\Component\Security\Core\User\PasswordUpgraderInterface;
 use Symfony\Component\Security\Core\User\UserProviderInterface;
 use Symfony\Component\Security\Http\Authentication\AuthenticationFailureHandlerInterface;
@@ -127,7 +128,7 @@ private function getCredentials(Request $request): array
         $credentials['username'] = trim($credentials['username']);
 
         if ('' === $credentials['username']) {
-            throw new BadRequestHttpException(sprintf('The key "%s" must be a non-empty string.', $this->options['username_parameter']));
+            throw new BadCredentialsException(sprintf('The key "%s" must be a non-empty string.', $this->options['username_parameter']));
         }
 
         $request->getSession()->set(SecurityRequestAttributes::LAST_USERNAME, $credentials['username']);
@@ -137,7 +138,7 @@ private function getCredentials(Request $request): array
         }
 
         if ('' === (string) $credentials['password']) {
-            throw new BadRequestHttpException(sprintf('The key "%s" must be a non-empty string.', $this->options['password_parameter']));
+            throw new BadCredentialsException(sprintf('The key "%s" must be a non-empty string.', $this->options['password_parameter']));
         }
 
         if (!\is_string($credentials['csrf_token'] ?? '') && (!\is_object($credentials['csrf_token']) || !method_exists($credentials['csrf_token'], '__toString'))) {

Tests/Authenticator/FormLoginAuthenticatorTest.php

@@ -44,7 +44,7 @@ protected function setUp(): void
 
     public function testHandleWhenUsernameEmpty()
     {
-        $this->expectException(BadRequestHttpException::class);
+        $this->expectException(BadCredentialsException::class);
         $this->expectExceptionMessage('The key "_username" must be a non-empty string.');
 
         $request = Request::create('/login_check', 'POST', ['_username' => '', '_password' => 's$cr$t']);
@@ -56,7 +56,7 @@ public function testHandleWhenUsernameEmpty()
 
     public function testHandleWhenPasswordEmpty()
     {
-        $this->expectException(BadRequestHttpException::class);
+        $this->expectException(BadCredentialsException::class);
         $this->expectExceptionMessage('The key "_password" must be a non-empty string.');
 
         $request = Request::create('/login_check', 'POST', ['_username' => 'foo', '_password' => '']);