chore: Add PR's requested changes

Author: matiasperrone-exo

app/Strategies/MFA/EmailOTPMFAChallengeStrategy.php

@@ -5,6 +5,7 @@
 use Auth\Repositories\IUserRecoveryCodeRepository;
 use Auth\User;
 use Models\OAuth2\Client;
+use Models\OAuth2\OAuth2OTP;
 use OAuth2\OAuth2Protocol;
 use OAuth2\Services\ITokenService;
 
@@ -34,13 +35,9 @@ public function issueChallenge(User $user, ?Client $client, bool $remember): arr
         ];
     }
 
-    public function verifyChallenge(User $user, string $code): void
+    public function verifyChallenge(User $user, string $code, ?Client $client = null): void
     {
-        $otp = $this->otp_repository->getByValueConnectionAndUserName(
-            $code,
-            OAuth2Protocol::OAuth2PasswordlessConnectionEmail,
-            $user->getEmail()
-        );
+        $otp = OAuth2OTP::fromParams($user->getEmail(), OAuth2Protocol::OAuth2PasswordlessConnectionEmail, $code);
 
         if (is_null($otp)) {
             throw new AuthenticationException("Non existent single-use code.");

app/Strategies/MFA/IMFAChallengeStrategy.php

@@ -6,7 +6,7 @@
 interface IMFAChallengeStrategy
 {
     public function issueChallenge(User $user, ?Client $client, bool $remember): array;
-    public function verifyChallenge(User $user, string $code): void;
+    public function verifyChallenge(User $user, string $code, ?Client $client = null): void;
     public function resendChallenge(User $user, ?Client $client, bool $remember): array;
     public function getPendingState(): ?array;
     public function clearPendingState(): void;