- WIP on v2.x authentication

src/Authorize/Bearer/JsonWebTokenDecoder.php

@@ -70,11 +70,11 @@ class JsonWebTokenDecoder
     public function isJWT() : bool
     {
         try {
-            return $this->parse();
+            return count(explode('.', $this->encoded)) === 3;
+        }
+        catch(\Throwable $t) {
+            return false;
         }
-        catch(\Throwable $t) {}
-
-        return false;
     }
 
     public function getPayload() : array

src/Authorize/Header/BearerMethod.php

@@ -6,6 +6,7 @@ use Psr\Http\Message\ServerRequestInterface;
 use Ulmus\User\Authorize\Bearer\JsonWebTokenDecoder;
 use Ulmus\User\Entity\UserInterface;
 use Ulmus\User\Lib\Authenticate;
+use Ulmus\User\Lib\AuthenticationMethodEnum;
 
 class BearerMethod implements MethodInterface
 {
@@ -23,10 +24,15 @@ class BearerMethod implements MethodInterface
 
         switch($this->autodetectTokenType()) {
             case BearerTokenTypeEnum::JsonWebToken:
+                $this->jwt->decode();
+
                 $payload = $this->jwt->getPayload();
 
                 if ( $payload['sub'] ?? false) {
-                    $request = $request->withAttribute('authentication_middleware:user_id', $payload['sub']);
+                    $request = $request
+                        ->withAttribute('authentication_middleware:method', AuthenticationMethodEnum::ForceLogin)
+                        ->withAttribute('authentication_middleware:jwt', $payload)
+                        ->withAttribute('authentication_middleware:user_id', $payload['sub']);
                 }
                 else {
                     throw new \InvalidArgumentException("Given JsonWebToken is missing a 'sub' key (which concords to user id)");
@@ -37,6 +43,8 @@ class BearerMethod implements MethodInterface
             case BearerTokenTypeEnum::UniqueKey:
                 # @TODO
                 break;
+
+            default:
         }
 
         return $request;

src/Authorize/HeaderAuthentication.php

@@ -62,7 +62,7 @@ class HeaderAuthentication implements AuthorizeMethodInterface
                 }
             }
         }*/
-
+        
         return $request->hasHeader('authorization');
     }
 }

src/Entity/User.php

@@ -48,12 +48,12 @@ abstract class User implements UserInterface {
     #[Field]
     public string $password;
 
-    #[Field\UpdatedAt(name: "updated_at", readonly: true)]
-    public ? Datetime $updatedAt;
-
     #[Field\CreatedAt(name: "created_at", readonly: true)]
     public Datetime $createdAt;
 
+    #[Field\UpdatedAt(name: "updated_at", readonly: true)]
+    public ? Datetime $updatedAt;
+
     public bool $logged = false;
 
     public function __toString() : string

src/Exception/InvalidUserException.php

@@ -0,0 +1,5 @@
+<?php
+
+namespace Ulmus\User\Exception;
+
+class InvalidUserException extends \Exception {}

src/Middleware/AuthenticationMiddleware.php

@@ -10,6 +10,7 @@ use Psr\Http\{
 };
 use Ulmus\User\Entity\UserInterface;
 use Ulmus\User\Authorize\PostRequestAuthentication;
+use Ulmus\User\Exception\InvalidUserException;
 use Ulmus\User\Lib\Authenticate;
 use Ulmus\User\Lib\AuthenticationMethodEnum;
 
@@ -23,28 +24,34 @@ class AuthenticationMiddleware implements MiddlewareInterface
     public function process(ServerRequestInterface $request, RequestHandlerInterface $handler): ResponseInterface
     {
         try {
-            if (null !== $id = $request->getAttribute('authentication_middleware:user_id')) {
-                $this->authenticator->loadUser($id);
-
-                if ( ! $this->authenticator->user->isLoaded() ) {
-                    throw new \Exception("Given user id do not match with an existing/active user");
-                }
-            }
-
-            switch($request->getAttribute('authentication_middleware:method')) {
-                case AuthenticationMethodEnum::ForceLogin:
-                    $this->authenticator->login();
-                    break;
-
-                case AuthenticationMethodEnum::UsernamePassword:
-                    $this->authenticator->authenticate($request->getAttribute('authentication_middleware:username'), $request->getAttribute('authentication_middleware:password'));
-                    break;
-            }
+            $this->launchAuthentication($request);
         }
-        catch(\Exception $e) {
+        catch(InvalidUserException $e) {
             return call_user_func($this->loginFailedResponse, [ 'api.error_message' => $e->getMessage() ]);
         }
 
         return $handler->handle($request);
     }
+
+    protected function launchAuthentication(ServerRequestInterface $request) : void
+    {
+        if (null !== $id = $request->getAttribute('authentication_middleware:user_id')) {
+
+            $this->authenticator->loadUser($id);
+
+            if ( ! $this->authenticator->user->isLoaded() ) {
+                throw new InvalidUserException("Given user id do not match with an existing/active user");
+            }
+        }
+
+        switch($request->getAttribute('authentication_middleware:method')) {
+            case AuthenticationMethodEnum::ForceLogin:
+                $this->authenticator->login();
+                break;
+
+            case AuthenticationMethodEnum::UsernamePassword:
+                $this->authenticator->authenticate($request->getAttribute('authentication_middleware:username'), $request->getAttribute('authentication_middleware:password'));
+                break;
+        }
+    }
 }