francis94c / ci-jwt

helpers/base64_helper.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
    *
    * @return string|bool          Decoded String.
    */
-  function base64url_decode(string $data, bool $strict=false) {
+  function base64url_decode(string $data, bool $strict = false) {
     $b64 = strtr($data, '-_', '+/');
     return base64_decode($b64, $strict);
   }

Braces +3 added lines, -1 removed lines

@@ -11,7 +11,9 @@
    */
   function base64url_encode(string $data) {
     $b64 = base64_encode($data);
-    if ($b64 === false) return false;
+    if ($b64 === false) {
+     return false;
+    }
     $url = strtr($b64, '+/', '-_');
     return rtrim($url, '=');
   }

libraries/JWT.php

Spacing +8 added lines, -8 removed lines

@@ -36,7 +36,7 @@ 
    */
   private $allow_unsigned = false;
 
-  function __construct($params=null) {
+  function __construct($params = null) {
     if ($params != null) $this->init($params);
     get_instance()->load->splint("francis94c/ci-jwt", "%base64");
   }
@@ -93,9 +93,9 @@ 
    * @param  [type] $secret [description]
    * @return [type]         [description]
    */
-  public function sign(string $secret=null):?string {
+  public function sign(string $secret = null): ?string {
     // Checks.
-    if  (count($this->payload) == 0) return null;
+    if (count($this->payload) == 0) return null;
     // $key is $secret.
     $key = $secret ?? $this->secret;
     $this->header["alg"] = $this->header["alg"] ?? self::HS256;
@@ -112,9 +112,9 @@ 
    * [token description]
    * @return string [description]
    */
-  public function token():?string {
+  public function token(): ?string {
     // Checks.
-    if  (count($this->payload) == 0) return null;
+    if (count($this->payload) == 0) return null;
     // Begin.
     $this->header["alg"] = self::NONE;
     return base64url_encode(json_encode($this->header)) . "." . base64url_encode(json_encode($this->payload)) . ".";
@@ -125,14 +125,14 @@ 
    * @param  string $secret [description]
    * @return bool           [description]
    */
-  public function verify(string $jwt, string $secret=null):bool {
+  public function verify(string $jwt, string $secret = null):bool {
     if (substr_count($jwt, ".") != 2) return false; // Invalid JWT.
     $key = $secret ?? $this->secret;
     $parts = explode(".", $jwt);
-    $header = json_decode(base64url_decode($parts[0]) ,true);
+    $header = json_decode(base64url_decode($parts[0]), true);
     if ($header == null) return false;
     $alg = $header["alg"] ?? self::HS256;
-    $payload = json_decode(base64url_decode($parts[1]) ,true);
+    $payload = json_decode(base64url_decode($parts[1]), true);
     if ($payload == null) return false;
     if ($parts[2] == "") {
       return $this->allow_unsigned;

Braces +31 added lines, -10 removed lines

@@ -37,7 +37,9 @@ 
   private $allow_unsigned = false;
 
   function __construct($params=null) {
-    if ($params != null) $this->init($params);
+    if ($params != null) {
+     $this->init($params);
+    }
     get_instance()->load->splint("francis94c/ci-jwt", "%base64");
   }
   /**
@@ -95,17 +97,25 @@ 
    */
   public function sign(string $secret=null):?string {
     // Checks.
-    if  (count($this->payload) == 0) return null;
+    if  (count($this->payload) == 0) {
+     return null;
+    }
     // $key is $secret.
     $key = $secret ?? $this->secret;
     $this->header["alg"] = $this->header["alg"] ?? self::HS256;
     $this->header["typ"] = $this->header["typ"] ?? self::JWT;
     $jwt = base64url_encode(json_encode($this->header));
-    if ($jwt === false) return null;
-    if ($jwt != "") $jwt .= ".";
+    if ($jwt === false) {
+     return null;
+    }
+    if ($jwt != "") {
+     $jwt .= ".";
+    }
     $payload = base64url_encode(json_encode($this->payload));
     $jwt .= $payload;
-    if ($key != "") return $this->sign_token($jwt, $key, $this->header["alg"]);
+    if ($key != "") {
+     return $this->sign_token($jwt, $key, $this->header["alg"]);
+    }
     return $jwt . ".";
   }
   /**
@@ -114,7 +124,9 @@ 
    */
   public function token():?string {
     // Checks.
-    if  (count($this->payload) == 0) return null;
+    if  (count($this->payload) == 0) {
+     return null;
+    }
     // Begin.
     $this->header["alg"] = self::NONE;
     return base64url_encode(json_encode($this->header)) . "." . base64url_encode(json_encode($this->payload)) . ".";
@@ -126,14 +138,21 @@ 
    * @return bool           [description]
    */
   public function verify(string $jwt, string $secret=null):bool {
-    if (substr_count($jwt, ".") != 2) return false; // Invalid JWT.
+    if (substr_count($jwt, ".") != 2) {
+     return false;
+    }
+    // Invalid JWT.
     $key = $secret ?? $this->secret;
     $parts = explode(".", $jwt);
     $header = json_decode(base64url_decode($parts[0]) ,true);
-    if ($header == null) return false;
+    if ($header == null) {
+     return false;
+    }
     $alg = $header["alg"] ?? self::HS256;
     $payload = json_decode(base64url_decode($parts[1]) ,true);
-    if ($payload == null) return false;
+    if ($payload == null) {
+     return false;
+    }
     if ($parts[2] == "") {
       return $this->allow_unsigned;
     }
@@ -162,7 +181,9 @@ 
    * @return string        Complete JWT.
    */
   private function sign_token(string $token, string $key, string $alg):string {
-    if ($alg == self::NONE) return $token . ".";
+    if ($alg == self::NONE) {
+     return $token . ".";
+    }
     $token = rtrim($token, ".");
     $signature = hash_hmac(self::ALGOS[$alg], $token, $key);
     return $token . "." . $signature;

phpunit/JWTTest.php

Spacing +1 added lines, -1 removed lines

@@ -20,7 +20,7 @@
    * @covers JWT::__construct
    */
   public static function setUpBeforeClass(): void {
-    self::$ci =& get_instance();
+    self::$ci = & get_instance();
     /**
      * [$params Config Items.]
      *