PHP 8.4 Support

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
     strategy:
       matrix:
         operating-system: ['ubuntu-latest']
-        php-versions: ['8.1', '8.2', '8.3']
+        php-versions: ['8.1', '8.2', '8.3', '8.4']
         phpunit-versions: ['latest']
     steps:
       - name: Checkout

.gitignore

@@ -1,3 +1,4 @@
 /.idea
+/.phpunit.result.cache
 /composer.lock
 /vendor

composer.json

@@ -30,8 +30,8 @@
     "paragonie/sodium_compat": "^1|^2"
   },
   "require-dev": {
-    "phpunit/phpunit": "^9",
-    "vimeo/psalm": "^4|^5"
+    "phpunit/phpunit": "^10",
+    "vimeo/psalm": "dev-master"
   },
   "scripts": {
     "full-test": [

src/Builder.php

@@ -50,9 +50,9 @@ class Builder extends PasetoBase
      * @throws PasetoException
      */
     public function __construct(
-        JsonToken $baseToken = null,
-        ProtocolInterface $protocol = null,
-        SendingKey $key = null
+        ?JsonToken $baseToken = null,
+        ?ProtocolInterface $protocol = null,
+        ?SendingKey $key = null
     ) {
         if (!$baseToken) {
             $baseToken = new JsonToken();
@@ -176,8 +176,8 @@ public function getImplicitAssertions(): array
      */
     public static function getLocal(
         SymmetricKey $key,
-        ProtocolInterface $version = null,
-        JsonToken $baseToken = null
+        ?ProtocolInterface $version = null,
+        ?JsonToken $baseToken = null
     ): self {
         if (!$version) {
             $version = $key->getProtocol();
@@ -202,8 +202,8 @@ public static function getLocal(
      */
     public static function getLocalWithKeyRing(
         SendingKeyRing    $key,
-        ProtocolInterface $version = null,
-        JsonToken         $baseToken = null
+        ?ProtocolInterface $version = null,
+        ?JsonToken         $baseToken = null
     ): self {
         if (!$version) {
             $version = $key->getProtocol();
@@ -228,8 +228,8 @@ public static function getLocalWithKeyRing(
      */
     public static function getPublic(
         AsymmetricSecretKey $key,
-        ProtocolInterface $version = null,
-        JsonToken $baseToken = null
+        ?ProtocolInterface $version = null,
+        ?JsonToken $baseToken = null
     ): self {
         if (!$version) {
             $version = $key->getProtocol();
@@ -254,8 +254,8 @@ public static function getPublic(
      */
     public static function getPublicWithKeyRing(
         SendingKeyRing    $key,
-        ProtocolInterface $version = null,
-        JsonToken         $baseToken = null
+        ?ProtocolInterface $version = null,
+        ?JsonToken         $baseToken = null
     ): self {
         if (!$version) {
             $version = $key->getProtocol();
@@ -308,7 +308,7 @@ public function setAudience(string $aud): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setExpiration(DateTimeInterface $time = null): self
+    public function setExpiration(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -348,7 +348,7 @@ public function setImplicitAssertions(array $assertions): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setIssuedAt(DateTimeInterface $time = null): self
+    public function setIssuedAt(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -384,7 +384,7 @@ public function setJti(string $id): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setNotBefore(DateTimeInterface $time = null): self
+    public function setNotBefore(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -569,7 +569,7 @@ public function setJsonToken(JsonToken $token): self
      *
      * @return self
      */
-    public function setVersion(ProtocolInterface $version = null): self
+    public function setVersion(?ProtocolInterface $version = null): self
     {
         if (!$version) {
             $version = new Version4();
@@ -615,6 +615,7 @@ public function toString(): string
                 ->format(DateTime::ATOM);
         }
         $claims = json_encode($claimsArray, JSON_FORCE_OBJECT);
+        assert(is_string($claims));
         $protocol = $this->version;
         ProtocolCollection::throwIfUnsupported($protocol);
 
@@ -716,7 +717,7 @@ public function withClaims(array $claims): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withExpiration(DateTimeInterface $time = null): self
+    public function withExpiration(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setExpiration($time);
     }
@@ -765,7 +766,7 @@ public function withImplicitAssertions(array $implicit): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withIssuedAt(DateTimeInterface $time = null): self
+    public function withIssuedAt(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setIssuedAt($time);
     }
@@ -798,7 +799,7 @@ public function withJti(string $id): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withNotBefore(DateTimeInterface $time = null): self
+    public function withNotBefore(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setNotBefore($time);
     }

src/Exception/PasetoException.php

@@ -7,6 +7,7 @@
 
 /**
  * Class PasetoException
+ * 
  * @package ParagonIE\Paseto\Exception
  */
 class PasetoException extends Exception
@@ -16,7 +17,7 @@ class PasetoException extends Exception
      * @param int $code
      * @param Throwable|null $previous
      */
-    public function __construct($message = "", $code = 0, Throwable $previous = null)
+    public function __construct($message = "", $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
         $this->setHelpfulMessage(ExceptionCode::explainErrorCode($code));

src/JsonToken.php

@@ -197,7 +197,7 @@ public function has(string $claim): bool
     {
         return array_key_exists($claim, $this->claims);
     }
-    
+
     /**
      * Set a claim to an arbitrary value.
      *
@@ -241,7 +241,7 @@ public function setClaims(array $claims): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setExpiration(DateTimeInterface $time = null): self
+    public function setExpiration(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -287,7 +287,7 @@ public function setFooterArray(array $footer = []): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setIssuedAt(DateTimeInterface $time = null): self
+    public function setIssuedAt(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -326,7 +326,7 @@ public function setJti(string $id): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function setNotBefore(DateTimeInterface $time = null): self
+    public function setNotBefore(?DateTimeInterface $time = null): self
     {
         if (!$time) {
             $time = new DateTime('NOW');
@@ -387,7 +387,7 @@ public function withClaims(array $claims): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withExpiration(DateTimeInterface $time = null): self
+    public function withExpiration(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setExpiration($time);
     }
@@ -422,7 +422,7 @@ public function withFooterArray(array $footer = []): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withIssuedAt(DateTimeInterface $time = null): self
+    public function withIssuedAt(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setIssuedAt($time);
     }
@@ -455,7 +455,7 @@ public function withJti(string $id): self
      * @param DateTimeInterface|null $time
      * @return self
      */
-    public function withNotBefore(DateTimeInterface $time = null): self
+    public function withNotBefore(?DateTimeInterface $time = null): self
     {
         return (clone $this)->setNotBefore($time);
     }

src/Keys/AsymmetricSecretKey.php

@@ -23,7 +23,7 @@ class AsymmetricSecretKey extends BaseAsymmetricSecretKey
 {
     public function __construct(
         string $keyData,
-        ProtocolInterface $protocol = null
+        ?ProtocolInterface $protocol = null
     ) {
         if (is_null($protocol)) {
             $protocol = new Version4();

src/Keys/Base/AsymmetricPublicKey.php

@@ -117,7 +117,10 @@ public static function v4(string $keyMaterial): V4AsymmetricPublicKey
      *
      * @throws Exception
      */
-    public static function newVersionKey(string $keyMaterial, ProtocolInterface $protocol = null): self
+    public static function newVersionKey(
+        string $keyMaterial,
+        ?ProtocolInterface $protocol = null,
+    ): self
     {
         $protocol = $protocol ?? new Version4();
 
@@ -155,7 +158,10 @@ abstract public function encodePem(): string;
      * @throws Exception
      * @throws TypeError
      */
-    public static function fromEncodedString(string $encoded, ProtocolInterface $version = null): self
+    public static function fromEncodedString(
+        string $encoded,
+        ?ProtocolInterface $version = null,
+    ): self
     {
         if (!$version) {
             $version = new Version4();
@@ -175,7 +181,7 @@ public static function fromEncodedString(string $encoded, ProtocolInterface $ver
      *
      * @throws Exception
      */
-    public static function importPem(string $pem, ProtocolInterface $protocol = null): self
+    public static function importPem(string $pem, ?ProtocolInterface $protocol = null): self
     {
         $protocol = $protocol ?? new Version4();
 

src/Keys/Base/AsymmetricSecretKey.php

@@ -159,7 +159,7 @@ public static function v4(string $keyMaterial): V4AsymmetricSecretKey
      * @throws Exception
      * @throws TypeError
      */
-    public static function generate(ProtocolInterface $protocol = null): self
+    public static function generate(?ProtocolInterface $protocol = null): self
     {
         $protocol = $protocol ?? new Version4;
         if (hash_equals($protocol::header(), Version3::HEADER)) {
@@ -178,7 +178,7 @@ public static function generate(ProtocolInterface $protocol = null): self
      *
      * @throws Exception
      */
-    public static function newVersionKey(string $keyMaterial, ProtocolInterface $protocol = null): self
+    public static function newVersionKey(string $keyMaterial, ?ProtocolInterface $protocol = null): self
     {
         $protocol = $protocol ?? new Version4();
 
@@ -216,7 +216,7 @@ abstract public function encodePem(): string;
      * @throws Exception
      * @throws TypeError
      */
-    public static function fromEncodedString(string $encoded, ProtocolInterface $version = null): self
+    public static function fromEncodedString(string $encoded, ?ProtocolInterface $version = null): self
     {
         if ($version && hash_equals($version::header(), Version3::HEADER)) {
             return V3AsymmetricSecretKey::fromEncodedString($encoded);
@@ -232,7 +232,7 @@ public static function fromEncodedString(string $encoded, ProtocolInterface $ver
      *
      * @throws Exception
      */
-    public static function importPem(string $pem, ProtocolInterface $protocol = null): self
+    public static function importPem(string $pem, ?ProtocolInterface $protocol = null): self
     {
         $protocol = $protocol ?? new Version4();
 

src/Keys/Base/SymmetricKey.php

@@ -45,7 +45,7 @@ class SymmetricKey implements ReceivingKey, SendingKey
      */
     public function __construct(
         string $keyMaterial,
-        ProtocolInterface $protocol = null
+        ?ProtocolInterface $protocol = null
     ) {
         $this->protocol = $protocol ?? new Version4;
 
@@ -81,7 +81,7 @@ public function __destruct()
      *
      * @throws Exception
      */
-    public static function generate(ProtocolInterface $protocol = null): self
+    public static function generate(?ProtocolInterface $protocol = null): self
     {
         $protocol = $protocol ?? new Version4;
         $length = $protocol::getSymmetricKeyByteLength();
@@ -172,7 +172,10 @@ public function encode(): string
      * @throws TypeError
      * @throws PasetoException
      */
-    public static function fromEncodedString(string $encoded, ProtocolInterface $version = null): self
+    public static function fromEncodedString(
+        string $encoded,
+        ?ProtocolInterface $version = null,
+    ): self
     {
         $decoded = Base64UrlSafe::decodeNoPadding($encoded);
         return new self($decoded, $version);

src/Keys/Version3/AsymmetricPublicKey.php

@@ -70,7 +70,10 @@ public function encodePem(): string
         );
     }
 
-    public static function fromEncodedString(string $encoded, ProtocolInterface $version = null): self
+    public static function fromEncodedString(
+        string $encoded,
+        ?ProtocolInterface $version = null,
+    ): self
     {
         $decodeString = Base64UrlSafe::decode($encoded);
         $length = Binary::safeStrlen($encoded);
@@ -105,7 +108,7 @@ public function toHexString(): string
      *
      * @throws Exception
      */
-    public static function importPem(string $pem, ProtocolInterface $protocol = null): self
+    public static function importPem(string $pem, ?ProtocolInterface $protocol = null): self
     {
         return new self($pem);
     }

src/Keys/Version3/AsymmetricSecretKey.php

@@ -36,7 +36,7 @@ public function __construct(
         parent::__construct($keyData, new Version3());
     }
 
-    public static function generate(ProtocolInterface $protocol = null): self
+    public static function generate(?ProtocolInterface $protocol = null): self
     {
         return new self(
             Util::dos2unix(SecretKey::generate(Version3::CURVE)->exportPem())
@@ -64,7 +64,7 @@ public function encodePem(): string
         return Util::dos2unix($this->key);
     }
 
-    public static function fromEncodedString(string $encoded, ProtocolInterface $version = null): self
+    public static function fromEncodedString(string $encoded, ?ProtocolInterface $version = null): self
     {
         $decoded = Base64UrlSafe::decodeNoPadding($encoded);
 
@@ -101,7 +101,7 @@ public function getPublicKey(): AsymmetricPublicKey
         );
     }
 
-    public static function importPem(string $pem, ProtocolInterface $protocol = null): self
+    public static function importPem(string $pem, ?ProtocolInterface $protocol = null): self
     {
         return new self($pem);
     }