Skip to content

Commit d2236db

Browse files
authored
Move to PHPv8 (#206)
* Bump to PHP 8.0 * Add version compatibility note to readme * Fix psalm error * Update composer.json --------- Co-authored-by: Marko Ivančić <[email protected]>
1 parent 0327c2f commit d2236db

File tree

4 files changed

+25
-11
lines changed

4 files changed

+25
-11
lines changed

.github/workflows/test.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
strategy:
1414
fail-fast: false
1515
matrix:
16-
php-versions: ["7.4", "8.0", "8.1"]
16+
php-versions: ["8.0", "8.1", "8.2"]
1717

1818
steps:
1919
- name: Setup PHP, with composer and extensions
@@ -55,7 +55,7 @@ jobs:
5555
run: composer install --no-progress --prefer-dist --optimize-autoloader
5656

5757
- name: Decide whether to run code coverage or not
58-
if: ${{ matrix.php-versions != '7.4' }}
58+
if: ${{ matrix.php-versions != '8.0' }}
5959
run: |
6060
echo "NO_COVERAGE=--no-coverage" >> $GITHUB_ENV
6161
@@ -65,7 +65,7 @@ jobs:
6565
./vendor/bin/phpunit $NO_COVERAGE
6666
6767
- name: Save coverage data
68-
if: ${{ matrix.php-versions == '7.4' }}
68+
if: ${{ matrix.php-versions == '8.0' }}
6969
uses: actions/upload-artifact@v1
7070
with:
7171
name: build-data
@@ -78,7 +78,7 @@ jobs:
7878
- name: Setup PHP, with composer and extensions
7979
uses: shivammathur/setup-php@v2 #https://github.com/shivammathur/setup-php
8080
with:
81-
php-version: "7.4"
81+
php-version: "8.0"
8282
extensions: mbstring, xml
8383
tools: composer:v2
8484
coverage: none
@@ -119,7 +119,7 @@ jobs:
119119
- name: Setup PHP, with composer and extensions
120120
uses: shivammathur/setup-php@v2 #https://github.com/shivammathur/setup-php
121121
with:
122-
php-version: "7.4"
122+
php-version: "8.0"
123123
extensions: mbstring, xml
124124
tools: composer:v2
125125
coverage: none
@@ -152,7 +152,7 @@ jobs:
152152
- name: Setup PHP, with composer and extensions
153153
uses: shivammathur/setup-php@v2 #https://github.com/shivammathur/setup-php
154154
with:
155-
php-version: "7.4"
155+
php-version: "8.0"
156156
tools: composer:v2
157157
extensions: mbstring, xml
158158

README.md

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,14 @@ Currently supported flows are:
1717

1818
![Main screen capture](docs/oidc.png)
1919

20+
## Version compatibility
21+
22+
| OIDC module | SimpleSAMLphp | PHP | Note |
23+
|:------------|:--------------|:------:|-----------------------------|
24+
| v4.\* | v2.\* | \>=8.0 | Recommended |
25+
| v3.\* | v2.\* | \>=7.4 | Abandoned from August 2023. |
26+
| v2.\* | v1.19.\* | \>=7.4 | |
27+
2028
## Installation
2129

2230
Installation can be as easy as executing:

composer.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,16 +17,16 @@
1717
}
1818
],
1919
"require": {
20-
"php": ">=7.4",
21-
"ext-curl": ">=7.4",
20+
"php": "^8.0",
21+
"ext-curl": "*",
2222
"ext-json": "*",
2323
"ext-openssl": "*",
2424
"ext-pdo": "*",
2525
"guzzlehttp/guzzle": "^7.0",
26-
"laminas/laminas-diactoros": "^2.2.1",
27-
"laminas/laminas-httphandlerrunner": "^1.1.0",
26+
"laminas/laminas-diactoros": "^2.25.2",
27+
"laminas/laminas-httphandlerrunner": "^2",
2828
"lcobucci/jwt": "^4.1",
29-
"league/oauth2-server": "^8.1.0",
29+
"league/oauth2-server": "^8.5.3",
3030
"nette/forms": "^3",
3131
"psr/container": "^1.0",
3232
"psr/log": "^1.1",

src/Form/ClientForm.php

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,12 @@ public function validateBackChannelLogoutUri(Form $form): void
102102
}
103103
}
104104

105+
/**
106+
* @param array $values
107+
* @param non-empty-string $regex
108+
* @param string $messagePrefix
109+
* @return void
110+
*/
105111
protected function validateByMatchingRegex(
106112
array $values,
107113
string $regex,

0 commit comments

Comments
 (0)