From 1a49537547c2da7dd371fb29f3d4da7c15e3596b Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Tue, 10 Mar 2026 21:27:08 +0100
Subject: [PATCH 01/10] Use local version of phplint

---
 .github/workflows/reusable_phplinter.yml | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/.github/workflows/reusable_phplinter.yml b/.github/workflows/reusable_phplinter.yml
index c7ce48c..26dc90b 100644
--- a/.github/workflows/reusable_phplinter.yml
+++ b/.github/workflows/reusable_phplinter.yml
@@ -43,13 +43,12 @@ jobs:
           tools: phive
           php-version: ${{ inputs.php-version }}
           coverage: "none"
+          tools: composer
+          # A conflict exists between overtrue/phplint and redis
           extensions: :redis
 
       - name: Install overtrue/phplint (v9.7)
-        run: |
-          sleep $((random % 10))
-          phive install overtrue/phplint@~9.7.1 --force-accept-unsigned --target ./bin
+        run: composer require overtrue/phplint
 
       - name: Lint PHP files
-        run: |
-          ./bin/phplint --no-cache --no-progress -v
+        run: vendor/bin/phplint --no-cache --no-progress -v

From e049a2251a372acad562f8b41f1cb9e440c2f031 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 12:23:12 +0100
Subject: [PATCH 02/10] Revert change to ArrayLogger

---
 src/ArrayLogger.php | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/ArrayLogger.php b/src/ArrayLogger.php
index 421df60..297e65b 100644
--- a/src/ArrayLogger.php
+++ b/src/ArrayLogger.php
@@ -7,8 +7,6 @@
 use SimpleSAML\Configuration;
 use SimpleSAML\Logger\LoggingHandlerInterface;
 
-use function array_merge;
-
 /**
  */
 class ArrayLogger implements LoggingHandlerInterface
@@ -28,7 +26,7 @@ public function __construct(Configuration $config)
 
     public function log(int $level, string $str): void
     {
-        $this->logs[$level] = array_merge($this->logs[$level], [$str]);
+        $this->logs[$level][] = $str;
     }
 
 

From bde8ac9931203ac4c5aff32102d6c91716229cf1 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 12:24:26 +0100
Subject: [PATCH 03/10] Fix linter-workflow

---
 .github/workflows/reusable_phplinter.yml | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/.github/workflows/reusable_phplinter.yml b/.github/workflows/reusable_phplinter.yml
index 26dc90b..e5f5c52 100644
--- a/.github/workflows/reusable_phplinter.yml
+++ b/.github/workflows/reusable_phplinter.yml
@@ -40,10 +40,9 @@ jobs:
       - name: Setup PHP runtime
         uses: shivammathur/setup-php@v2
         with:
-          tools: phive
+          tools: composer, phive
           php-version: ${{ inputs.php-version }}
           coverage: "none"
-          tools: composer
           # A conflict exists between overtrue/phplint and redis
           extensions: :redis
 

From ae5875aba69a159df0fdfaf8491201b0c1cf73e1 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 12:29:20 +0100
Subject: [PATCH 04/10] Fix composer-cache

---
 .github/workflows/php.yml | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/php.yml b/.github/workflows/php.yml
index 5f82b6f..d4f17a0 100644
--- a/.github/workflows/php.yml
+++ b/.github/workflows/php.yml
@@ -77,7 +77,7 @@ jobs:
       - name: Cache composer dependencies
         uses: actions/cache@v5
         with:
-          path: $COMPOSER_CACHE
+          path: ${{ env.COMPOSER_CACHE }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: ${{ runner.os }}-composer-
 
@@ -140,7 +140,7 @@ jobs:
       - name: Cache composer dependencies
         uses: actions/cache@v5
         with:
-          path: $COMPOSER_CACHE
+          path: ${{ env.COMPOSER_CACHE }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: ${{ runner.os }}-composer-
 
@@ -177,7 +177,7 @@ jobs:
       - name: Cache composer dependencies
         uses: actions/cache@v5
         with:
-          path: $COMPOSER_CACHE
+          path: ${{ env.COMPOSER_CACHE }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: ${{ runner.os }}-composer-
 
@@ -241,7 +241,7 @@ jobs:
       - name: Cache composer dependencies
         uses: actions/cache@v5
         with:
-          path: $COMPOSER_CACHE
+          path: ${{ env.COMPOSER_CACHE }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: ${{ runner.os }}-composer-
 

From 47f5bed855ba48b06cc5c467f6a43aa777c985e9 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 12:37:16 +0100
Subject: [PATCH 05/10] Delete 1

---
 1 | 35 -----------------------------------
 1 file changed, 35 deletions(-)
 delete mode 100644 1

diff --git a/1 b/1
deleted file mode 100644
index f13d063..0000000
--- a/1
+++ /dev/null
@@ -1,35 +0,0 @@
-pick a6e6d49 Add support for PHPunit 13
-pick e0061f5 Move contents of lib-dir to src-dir
-pick 61fddec Fix CI
-s 08712a5 Fix CI
-
-# Rebase 720ccf7..08712a5 onto 720ccf7 (4 commands)
-#
-# Commands:
-# p, pick <commit> = use commit
-# r, reword <commit> = use commit, but edit the commit message
-# e, edit <commit> = use commit, but stop for amending
-# s, squash <commit> = use commit, but meld into previous commit
-# f, fixup [-C | -c] <commit> = like "squash" but keep only the previous
-#                    commit's log message, unless -C is used, in which case
-#                    keep only this commit's message; -c is same as -C but
-#                    opens the editor
-# x, exec <command> = run command (the rest of the line) using shell
-# b, break = stop here (continue rebase later with 'git rebase --continue')
-# d, drop <commit> = remove commit
-# l, label <label> = label current HEAD with a name
-# t, reset <label> = reset HEAD to a label
-# m, merge [-C <commit> | -c <commit>] <label> [# <oneline>]
-#         create a merge commit using the original merge commit's
-#         message (or the oneline, if no original merge commit was
-#         specified); use -c <commit> to reword the commit message
-# u, update-ref <ref> = track a placeholder for the <ref> to be updated
-#                       to this position in the new commits. The <ref> is
-#                       updated at the end of the rebase
-#
-# These lines can be re-ordered; they are executed from top to bottom.
-#
-# If you remove a line here THAT COMMIT WILL BE LOST.
-#
-# However, if you remove everything, the rebase will be aborted.
-#

From 3d0a3e2972284148331bdb7acac5daeb95f84bd6 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 12:44:58 +0100
Subject: [PATCH 06/10] Fix tests

---
 .phpunit.cache/test-results                | 1 +
 {src => tests/src}/ReduceSpillOverTest.php | 3 ++-
 2 files changed, 3 insertions(+), 1 deletion(-)
 create mode 100644 .phpunit.cache/test-results
 rename {src => tests/src}/ReduceSpillOverTest.php (93%)

diff --git a/.phpunit.cache/test-results b/.phpunit.cache/test-results
new file mode 100644
index 0000000..5ca1938
--- /dev/null
+++ b/.phpunit.cache/test-results
@@ -0,0 +1 @@
+{"version":2,"defects":{"SimpleSAML\\Test\\TestUtils\\InMemoryStoreTest::testState":8,"SimpleSAML\\Test\\TestUtils\\InMemoryStoreTest::testExpiration":8},"times":{"SimpleSAML\\Test\\TestUtils\\InMemoryStoreTest::testState":0.004,"SimpleSAML\\Test\\TestUtils\\InMemoryStoreTest::testExpiration":2.001,"SimpleSAML\\Test\\TestUtils\\ReduceSpillOverTest::testSetState":0.013,"SimpleSAML\\Test\\TestUtils\\ReduceSpillOverTest::testStateRemoved":0.017}}
\ No newline at end of file
diff --git a/src/ReduceSpillOverTest.php b/tests/src/ReduceSpillOverTest.php
similarity index 93%
rename from src/ReduceSpillOverTest.php
rename to tests/src/ReduceSpillOverTest.php
index de0ad44..3dfad99 100644
--- a/src/ReduceSpillOverTest.php
+++ b/tests/src/ReduceSpillOverTest.php
@@ -2,10 +2,11 @@
 
 declare(strict_types=1);
 
-namespace SimpleSAML\TestUtils;
+namespace SimpleSAML\Test\TestUtils;
 
 use SimpleSAML\Configuration;
 use SimpleSAML\Error\ConfigurationError;
+use SimpleSAML\TestUtils\ClearStateTestCase;
 
 /**
  * Test that ensures state doesn't spill over between tests

From ed52c9077af1965c832c749e7ed062408722f6f7 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Sat, 14 Mar 2026 21:43:16 +0100
Subject: [PATCH 07/10] Final SSP 2.5

---
 composer.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/composer.json b/composer.json
index 37eee22..a65d742 100644
--- a/composer.json
+++ b/composer.json
@@ -33,7 +33,7 @@
         "phpstan/phpstan-phpunit": "^2.0",
         "phpunit/phpunit": "^12.3 || ^13.0",
         "psr/log": "^3.0",
-        "simplesamlphp/simplesamlphp": "^2.5@dev",
+        "simplesamlphp/simplesamlphp": "^2.5",
         "slevomat/coding-standard": "^8.25",
         "squizlabs/php_codesniffer": "^4.0",
         "symfony/phpunit-bridge": "^8.0",

From 40f1bfad812fbeb4176b3b79ee525d0c8e1ca89a Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Mon, 16 Mar 2026 11:41:25 +0100
Subject: [PATCH 08/10] Exclude vendor-directory

---
 .github/workflows/reusable_phplinter.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/reusable_phplinter.yml b/.github/workflows/reusable_phplinter.yml
index e5f5c52..39c8395 100644
--- a/.github/workflows/reusable_phplinter.yml
+++ b/.github/workflows/reusable_phplinter.yml
@@ -50,4 +50,4 @@ jobs:
         run: composer require overtrue/phplint
 
       - name: Lint PHP files
-        run: vendor/bin/phplint --no-cache --no-progress -v
+        run: vendor/bin/phplint --exclude=vendor --no-cache --no-progress -v

From 341f8ea45c1818459c5f8e555ff6eb3646d14b17 Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Mon, 16 Mar 2026 11:42:20 +0100
Subject: [PATCH 09/10] Bump phplinter

---
 .github/workflows/php.yml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/php.yml b/.github/workflows/php.yml
index d4f17a0..91ea6af 100644
--- a/.github/workflows/php.yml
+++ b/.github/workflows/php.yml
@@ -20,7 +20,7 @@ jobs:
       matrix:
         php-version: ['8.3', '8.4', '8.5']
 
-    uses: simplesamlphp/simplesamlphp-test-framework/.github/workflows/reusable_phplinter.yml@v1.11.1
+    uses: simplesamlphp/simplesamlphp-test-framework/.github/workflows/reusable_phplinter.yml@v1.11.3
     with:
       php-version: ${{ matrix.php-version }}
 
@@ -29,7 +29,7 @@ jobs:
     strategy:
       fail-fast: false
 
-    uses: simplesamlphp/simplesamlphp-test-framework/.github/workflows/reusable_linter.yml@v1.11.1
+    uses: simplesamlphp/simplesamlphp-test-framework/.github/workflows/reusable_linter.yml@v1.11.3
     with:
       enable_eslinter: false
       enable_jsonlinter: true

From 739f42fa3d75204b43079acecb4416ae0a8ebbad Mon Sep 17 00:00:00 2001
From: Tim van Dijen <tvdijen@gmail.com>
Date: Mon, 16 Mar 2026 13:49:02 +0100
Subject: [PATCH 10/10] Don't worry about platform requirements when linting

---
 .github/workflows/reusable_phplinter.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/reusable_phplinter.yml b/.github/workflows/reusable_phplinter.yml
index 39c8395..6fe6418 100644
--- a/.github/workflows/reusable_phplinter.yml
+++ b/.github/workflows/reusable_phplinter.yml
@@ -47,7 +47,7 @@ jobs:
           extensions: :redis
 
       - name: Install overtrue/phplint (v9.7)
-        run: composer require overtrue/phplint
+        run: composer require overtrue/phplint --ignore-platform-reqs
 
       - name: Lint PHP files
         run: vendor/bin/phplint --exclude=vendor --no-cache --no-progress -v