From aeed749238a59f68b574b3e509e959ac5bcef8d4 Mon Sep 17 00:00:00 2001 From: Filippo Tessarotto Date: Fri, 23 Dec 2022 13:12:56 +0100 Subject: [PATCH] Rename expirationTime/lifetime to what it really is: idle timeout --- docs/configuration.md | 8 ++++---- src/Storageless/Http/SessionMiddleware.php | 12 ++++++------ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/docs/configuration.md b/docs/configuration.md index d6bf24b1..e9a80d7f 100644 --- a/docs/configuration.md +++ b/docs/configuration.md @@ -14,7 +14,7 @@ use PSR7Sessions\Storageless\Http\SessionMiddleware; $sessionMiddleware = SessionMiddleware::fromSymmetricKeyDefaults( InMemory::base64Encoded('OpcMuKmoxkhzW0Y1iESpjWwL/D3UBdDauJOe742BJ5Q='), // replace this with a key of your own (see below) - 1200 // session lifetime, in seconds + 1200 // session idle timeout, in seconds ); ``` @@ -35,7 +35,7 @@ use PSR7Sessions\Storageless\Http\SessionMiddleware; $sessionMiddleware = SessionMiddleware::fromRsaAsymmetricKeyDefaults( InMemory::file('/path/to/private_key.pem'), InMemory::file('/path/to/public_key.pem'), - 1200 // session lifetime, in seconds + 1200 // session idle timeout, in seconds ); ``` @@ -83,7 +83,7 @@ $sessionMiddleware = new SessionMiddleware( InMemory::base64Encoded('FW6ELbU3817sflp2XmnCQ3JJTIMihR1RctQu7VQ73Fg=') ), SessionMiddleware::buildDefaultCookie(), - 1200, // session lifetime, in seconds + 1200, // session idle timeout, in seconds SystemClock::fromSystemTimezone(), 60 // session automatic refresh time, in seconds ); @@ -137,7 +137,7 @@ return new SessionMiddleware( ->withSecure(false) ->withHttpOnly(true) ->withPath('/'), - 1200, // session lifetime, in seconds + 1200, // session idle timeout, in seconds SystemClock::fromUTC(), ); ``` diff --git a/src/Storageless/Http/SessionMiddleware.php b/src/Storageless/Http/SessionMiddleware.php index ba1b829d..7609afaa 100644 --- a/src/Storageless/Http/SessionMiddleware.php +++ b/src/Storageless/Http/SessionMiddleware.php @@ -61,7 +61,7 @@ final class SessionMiddleware implements MiddlewareInterface public function __construct( Configuration $configuration, SetCookie $defaultCookie, - private int $expirationTime, + private int $idleTimeout, private Clock $clock, private int $refreshTime = self::DEFAULT_REFRESH_TIME, ) { @@ -72,7 +72,7 @@ public function __construct( /** * This constructor simplifies instantiation when using HTTPS (REQUIRED!) and symmetric key encryption */ - public static function fromSymmetricKeyDefaults(Signer\Key $symmetricKey, int $expirationTime): self + public static function fromSymmetricKeyDefaults(Signer\Key $symmetricKey, int $idleTimeout): self { return new self( Configuration::forSymmetricSigner( @@ -80,7 +80,7 @@ public static function fromSymmetricKeyDefaults(Signer\Key $symmetricKey, int $e $symmetricKey, ), self::buildDefaultCookie(), - $expirationTime, + $idleTimeout, new SystemClock(new DateTimeZone(date_default_timezone_get())), ); } @@ -92,7 +92,7 @@ public static function fromSymmetricKeyDefaults(Signer\Key $symmetricKey, int $e public static function fromRsaAsymmetricKeyDefaults( Signer\Key $privateRsaKey, Signer\Key $publicRsaKey, - int $expirationTime, + int $idleTimeout, ): self { return new self( Configuration::forAsymmetricSigner( @@ -101,7 +101,7 @@ public static function fromRsaAsymmetricKeyDefaults( $publicRsaKey, ), self::buildDefaultCookie(), - $expirationTime, + $idleTimeout, new SystemClock(new DateTimeZone(date_default_timezone_get())), ); } @@ -222,7 +222,7 @@ private function shouldTokenBeRefreshed(Token|null $token): bool private function getTokenCookie(SessionInterface $sessionContainer): SetCookie { $now = $this->clock->now(); - $expiresAt = $now->add(new DateInterval(sprintf('PT%sS', $this->expirationTime))); + $expiresAt = $now->add(new DateInterval(sprintf('PT%sS', $this->idleTimeout))); return $this ->defaultCookie