fix: check kid before using as array index

firebase · Nov 1, 2022 · bad1b04 · bad1b04
1 parent ddfaddc
commit bad1b04
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/JWT.php b/src/JWT.php
@@ -425,14 +425,15 @@ private static function getKey(
             return $keyOrKeyArray;
         }
 
+        if (empty($kid)) {
+            throw new UnexpectedValueException('"kid" empty, unable to lookup correct key');
+        }
+
         if ($keyOrKeyArray instanceof CachedKeySet) {
             // Skip "isset" check, as this will automatically refresh if not set
             return $keyOrKeyArray[$kid];
         }
 
-        if (empty($kid)) {
-            throw new UnexpectedValueException('"kid" empty, unable to lookup correct key');
-        }
         if (!isset($keyOrKeyArray[$kid])) {
             throw new UnexpectedValueException('"kid" invalid, unable to lookup correct key');
         }