fix: use short classname for config()

If FQCN is specified, devs cannot override.
codeigniter4 · Aug 24, 2023 · 23286e1 · 23286e1
1 parent 970c67e
commit 23286e1
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 8 deletions.
diff --git a/src/Authentication/Authenticators/JWT.php b/src/Authentication/Authenticators/JWT.php
@@ -61,7 +61,8 @@ public function __construct(UserModel $provider)
      */
     public function attempt(array $credentials): Result
     {
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         /** @var IncomingRequest $request */
         $request = service('request');
@@ -142,7 +143,7 @@ public function check(array $credentials): Result
                 'success' => false,
                 'reason'  => lang(
                     'Auth.noToken',
-                    [config(AuthJWT::class)->authenticatorHeader]
+                    [config('AuthJWT')->authenticatorHeader]
                 ),
             ]);
         }
@@ -196,7 +197,8 @@ public function loggedIn(): bool
         /** @var IncomingRequest $request */
         $request = service('request');
 
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         return $this->attempt([
             'token' => $request->getHeaderLine($config->authenticatorHeader),

diff --git a/src/Authentication/JWT/Adapters/FirebaseAdapter.php b/src/Authentication/JWT/Adapters/FirebaseAdapter.php
@@ -79,7 +79,8 @@ public function decode(string $encodedToken, $keyset): stdClass
      */
     private function createKeysForDecode($keyset)
     {
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         $configKeys = $config->keys[$keyset];
 
@@ -127,7 +128,8 @@ public function encode(array $payload, $keyset, ?array $headers = null): string
      */
     private function createKeysForEncode($keyset): array
     {
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         if (isset($config->keys[$keyset][0]['secret'])) {
             $key = $config->keys[$keyset][0]['secret'];

diff --git a/src/Authentication/JWT/JWSEncoder.php b/src/Authentication/JWT/JWSEncoder.php
@@ -39,7 +39,8 @@ public function encode(
             'Cannot pass $claims[\'exp\'] and $ttl at the same time.'
         );
 
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         $payload = array_merge(
             $config->defaultClaims,

diff --git a/src/Filters/JWTAuth.php b/src/Filters/JWTAuth.php
@@ -59,7 +59,8 @@ private function getTokenFromHeader(RequestInterface $request): string
     {
         assert($request instanceof IncomingRequest);
 
-        $config = config(AuthJWT::class);
+        /** @var AuthJWT $config */
+        $config = config('AuthJWT');
 
         $tokenHeader = $request->getHeaderLine(
             $config->authenticatorHeader ?? 'Authorization'

diff --git a/src/Models/BaseModel.php b/src/Models/BaseModel.php
@@ -20,7 +20,7 @@ abstract class BaseModel extends Model
 
     public function __construct()
     {
-        $this->authConfig = config(Auth::class);
+        $this->authConfig = config('Auth');
 
         if ($this->authConfig->DBGroup !== null) {
             $this->DBGroup = $this->authConfig->DBGroup;