chore: Update composer-deps

brainbits · Jan 17, 2025 · 4c571b0 · 4c571b0
1 parent 50f726d
commit 4c571b0
Show file tree

Hide file tree

Showing 8 changed files with 46 additions and 36 deletions.
diff --git a/.github/workflows/test-redis.yml b/.github/workflows/test-redis.yml
@@ -14,8 +14,8 @@ jobs:
       matrix:
         dependencies: ["lowest", "highest"]
         php-version:
-          - "8.2"
           - "8.3"
+          - "8.4"
         operating-system: ["ubuntu-latest"]
 
     steps:

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -14,8 +14,8 @@ jobs:
       matrix:
         dependencies: ["lowest", "highest"]
         php-version:
-          - "8.2"
           - "8.3"
+          - "8.4"
         operating-system: ["ubuntu-latest"]
 
     steps:

diff --git a/composer.json b/composer.json
@@ -11,17 +11,17 @@
         }
     ],
     "require": {
-        "php": "^8.2",
+        "php": "^8.3",
         "psr/clock": "^1.0"
     },
     "require-dev": {
         "brainbits/phpcs-standard": "^7.0.1",
-        "brainbits/phpstan-rules": "^3.1",
+        "brainbits/phpstan-rules": "^4.0",
         "matthiasnoback/symfony-config-test": "^5.1",
         "matthiasnoback/symfony-dependency-injection-test": "^6.0",
         "mikey179/vfsstream": "^1.6.11",
-        "phpstan/phpstan": "^1.10.67",
-        "phpunit/phpunit": "^11.3.6",
+        "phpstan/phpstan": "^2.1.1",
+        "phpunit/phpunit": "^11.5",
         "predis/predis": "^2.2",
         "symfony/clock": "^6.4|^7.0",
         "symfony/config": "^6.4|^7.0",
@@ -30,8 +30,8 @@
         "symfony/http-kernel": "^6.4|^7.0",
         "symfony/routing": "^6.4|^7.0",
         "symfony/security-core": "^6.4|^7.0",
-        "phpstan/phpstan-phpunit": "^1.3",
-        "phpstan/phpstan-symfony": "^1.3"
+        "phpstan/phpstan-phpunit": "^2.0",
+        "phpstan/phpstan-symfony": "^2.0"
     },
     "suggest": {
         "predis/predis": "If you want to use the PredisStorage",

diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -4,6 +4,9 @@ parameters:
         - src
     bootstrapFiles:
         - vendor/autoload.php
+    excludePaths:
+        # Declaring types for static function parameters is not possible
+        - src/Bundle/DependencyInjection/Configuration.php
 includes:
     - vendor/brainbits/phpstan-rules/rules.neon
     - vendor/phpstan/phpstan-phpunit/rules.neon

diff --git a/src/Bundle/DependencyInjection/BrainbitsBlockingExtension.php b/src/Bundle/DependencyInjection/BrainbitsBlockingExtension.php
@@ -20,7 +20,6 @@
 use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
 
-use function assert;
 use function sprintf;
 
 class BrainbitsBlockingExtension extends Extension
@@ -32,8 +31,11 @@ public function load(array $configs, ContainerBuilder $container): void
         $yamlLoader->load('services.yaml');
         $configuration = $this->getConfiguration($configs, $container);
 
-        assert($configuration instanceof Configuration);
-
+        /** @var array{
+         *     storage: non-empty-array<string>,
+         *     clock?: class-string,
+         *     block_interval: int,
+         *     owner_factory: array{value?: string, driver: string, service: string}} $config */
         $config = $this->processConfiguration($configuration, $configs);
 
         if (isset($config['storage']['predis'])) {

diff --git a/src/Owner/SymfonySessionOwnerFactory.php b/src/Owner/SymfonySessionOwnerFactory.php
@@ -16,7 +16,6 @@
 use Brainbits\Blocking\Exception\NoSessionException;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\RequestStack;
-use Symfony\Component\HttpFoundation\Session\SessionInterface;
 
 final readonly class SymfonySessionOwnerFactory implements OwnerFactoryInterface
 {
@@ -32,9 +31,6 @@ public function createOwner(): Owner
         }
 
         $session = $request->getSession();
-        if (!$session instanceof SessionInterface) {
-            throw NoSessionException::create();
-        }
 
         return new Owner($session->getId());
     }

diff --git a/src/Storage/FilesystemStorage.php b/src/Storage/FilesystemStorage.php
@@ -22,13 +22,13 @@
 use Brainbits\Blocking\Owner\Owner;
 use DateTimeImmutable;
 use Psr\Clock\ClockInterface;
+use Throwable;
 
 use function assert;
 use function dirname;
 use function file_exists;
 use function file_get_contents;
 use function file_put_contents;
-use function is_array;
 use function is_string;
 use function is_writable;
 use function json_decode;
@@ -137,13 +137,16 @@ public function exists(BlockIdentity $identity): bool
         $metaContent = file_get_contents($metaFilename);
         assert(is_string($metaContent));
         assert($metaContent !== '');
+        /** @var array{ttl: int, updatedAt: string} $metaData */
         $metaData = json_decode($metaContent, true);
-        assert(is_array($metaData));
 
         $now = $this->clock->now();
-
-        $expiresAt = (new DateTimeImmutable((string) $metaData['updatedAt'], $now->getTimezone()))
-            ->modify('+' . $metaData['ttl'] . ' seconds');
+        try {
+            $expiresAt = (new DateTimeImmutable((string) $metaData['updatedAt'], $now->getTimezone()))
+                ->modify('+' . $metaData['ttl'] . ' seconds');
+        } catch (Throwable) {
+            throw UnserializeFailedException::createFromInput($metaContent);
+        }
 
         return $expiresAt > $now;
     }
@@ -162,16 +165,19 @@ public function get(BlockIdentity $identity): Block|null
             throw UnserializeFailedException::createFromInput($content);
         }
 
+        /** @var array{identity: string, owner: string} $data */
         $data = json_decode($content, true);
 
-        assert(is_array($data));
-        assert($data['identity'] ?? false);
-        assert($data['owner'] ?? false);
+        try {
+            $block = new Block(
+                new BlockIdentity($data['identity']),
+                new Owner($data['owner']),
+            );
+        } catch (Throwable) {
+            throw UnserializeFailedException::createFromInput($content);
+        }
 
-        return new Block(
-            new BlockIdentity($data['identity']),
-            new Owner($data['owner']),
-        );
+        return $block;
     }
 
     private function getFilename(BlockIdentity $identifier): string

diff --git a/src/Storage/PredisStorage.php b/src/Storage/PredisStorage.php
@@ -15,13 +15,13 @@
 
 use Brainbits\Blocking\Block;
 use Brainbits\Blocking\Exception\IOException;
+use Brainbits\Blocking\Exception\UnserializeFailedException;
 use Brainbits\Blocking\Identity\BlockIdentity;
 use Brainbits\Blocking\Owner\Owner;
 use Predis\ClientInterface;
 use Predis\PredisException;
+use Throwable;
 
-use function assert;
-use function is_array;
 use function json_decode;
 use function json_encode;
 
@@ -110,16 +110,19 @@ public function get(BlockIdentity $identity): Block|null
             throw IOException::getFailed((string) $identity);
         }
 
+        /** @var array{identity: string, owner: string} $data */
         $data = json_decode($content, true);
 
-        assert(is_array($data));
-        assert($data['identity'] ?? false);
-        assert($data['owner'] ?? false);
+        try {
+            $block = new Block(
+                new BlockIdentity($data['identity']),
+                new Owner($data['owner']),
+            );
+        } catch (Throwable) {
+            throw UnserializeFailedException::createFromInput($content);
+        }
 
-        return new Block(
-            new BlockIdentity($data['identity']),
-            new Owner($data['owner']),
-        );
+        return $block;
     }
 
     private function createKey(BlockIdentity $identity): string
-Original file line number
+Diff line change
@@ Expand Up / @@ -14,8 +14,8 @@ jobs: @@
           matrix:
             dependencies: ["lowest", "highest"]
             php-version:
-              - "8.2"
               - "8.3"
+              - "8.4"
             operating-system: ["ubuntu-latest"]
         steps:
@@ Expand Down @@