Skip to content

Commit 0550086

Browse files
committed
Merge branch 'version-6' into wip-version-7
2 parents a453d01 + b0dfe70 commit 0550086

File tree

4 files changed

+16
-10
lines changed

4 files changed

+16
-10
lines changed

.github/workflows/test.yaml

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -208,6 +208,10 @@ jobs:
208208

209209
conformance-suite:
210210
runs-on: ubuntu-latest
211+
strategy:
212+
fail-fast: false
213+
matrix:
214+
ssp-version: ["v2.3.7", "v2.4.2"]
211215
env:
212216
SUITE_BASE_URL: https://localhost.emobix.co.uk:8443
213217
VERSION: release-v5.1.35
@@ -240,7 +244,7 @@ jobs:
240244
working-directory: ./main
241245
# Must run after conformance suite since they share a docker network.
242246
run: |
243-
OIDC_VERSION=@dev docker compose -f docker/docker-compose.yml --project-directory . up -d --build
247+
SSP_VERSION=${{ matrix.ssp-version }} OIDC_VERSION=@dev docker compose -f docker/docker-compose.yml --project-directory . up -d --build
244248
sleep 30
245249
# while ! curl -skfail https://op.local.stack-dev.cirrusidentity.com/.well-known/openid-configuration >/dev/null; do sleep 2; done
246250
- name: Run Basic conformance tests

README.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -39,13 +39,13 @@ during module development. SimpleSAMLphp has followed semantic versioning for it
3939
for example, that v5.* of the OIDC module should work with any v2.* of SimpleSAMLphp. However, please note that
4040
PHP version requirements have changed in minor SimpleSAMLphp releases.
4141

42-
| OIDC module | Tested SimpleSAMLphp | PHP | Note |
43-
|:------------|:---------------------|:------:|-----------------------------|
44-
| v6.\* | v2.3.\* | \>=8.2 | Recommended |
45-
| v5.\* | v2.1.\* | \>=8.1 | |
46-
| v4.\* | v2.0.\* | \>=8.0 | |
47-
| v3.\* | v2.0.\* | \>=7.4 | Abandoned from August 2023. |
48-
| v2.\* | v1.19.\* | \>=7.4 | |
42+
| OIDC module | Tested SimpleSAMLphp | PHP | Note |
43+
|:------------|:---------------------|:------:|-------------|
44+
| v6.\* | v2.3.\*, v2.4.\* | \>=8.2 | Recommended |
45+
| v5.\* | v2.1.\* | \>=8.1 | |
46+
| v4.\* | v2.0.\* | \>=8.0 | |
47+
| v3.\* | v2.0.\* | \>=7.4 | |
48+
| v2.\* | v1.19.\* | \>=7.4 | |
4949

5050
### Upgrading?
5151

docker/Dockerfile

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
1-
FROM cirrusid/simplesamlphp:v2.3.7
2-
#FROM cicnavi/simplesamlphp:dev
1+
ARG SSP_VERSION="v2.4.2"
2+
FROM cirrusid/simplesamlphp:${SSP_VERSION}
3+
#FROM cicnavi/simplesamlphp:${SSP_VERSION}
34

45
RUN apt-get update && apt-get install -y sqlite3
56
# Prepopulate the DB with items needed for testing

docker/docker-compose.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ services:
1616
context: .
1717
dockerfile: docker/Dockerfile
1818
args:
19+
SSP_VERSION: "${SSP_VERSION}"
1920
OIDC_VERSION: "${OIDC_VERSION}"
2021
environment:
2122
- STAGINGCOMPOSERREPOS=oidc

0 commit comments

Comments
 (0)