Skip to content

Commit 113cfca

Browse files
authored
Merge pull request #394 from open-zaak/chore/oaf-218-ci-hardening
🔒 [maykinmedia/open-api-framework#218] Add zizmor action and fix warnings/errors
2 parents 09b6b71 + 066e2f8 commit 113cfca

8 files changed

Lines changed: 25 additions & 20 deletions

File tree

.github/workflows/ci.yml

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -45,13 +45,16 @@ jobs:
4545
options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
4646

4747
steps:
48-
- uses: actions/checkout@v4
49-
- uses: actions/setup-python@v5
48+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
49+
with:
50+
persist-credentials: false
51+
- uses: actions/setup-python@a309ff8b426b58ec0e2a45f0f869d46889d02405 # v6.2.0
5052
with:
5153
python-version: "3.12"
52-
- uses: actions/setup-node@v4
54+
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
5355
with:
5456
node-version: "24"
57+
package-manager-cache: false
5558

5659
- name: Start CI docker compose
5760
run: docker compose --file docker-compose.ssl.yml up --detach mock
@@ -72,7 +75,7 @@ jobs:
7275
DB_POOL_ENABLED: ${{ matrix.use_pooling }}
7376

7477
- name: Publish coverage report
75-
uses: codecov/codecov-action@v4
78+
uses: codecov/codecov-action@57e3a136b779b570ffcdbf80b3bdc90e7fab3de2 # v6.0.0
7679
with:
7780
token: ${{ secrets.CODECOV_TOKEN }}
7881

@@ -81,9 +84,11 @@ jobs:
8184
name: Documentation build
8285

8386
steps:
84-
- uses: actions/checkout@v4
87+
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
88+
with:
89+
persist-credentials: false
8590
- name: Set up backend environment
86-
uses: maykinmedia/setup-django-backend@v1.3
91+
uses: maykinmedia/setup-django-backend@a9abe0987130ed667fa09ce177a5ae0bd153aed1 # v1.3
8792
with:
8893
python-version: "3.12"
8994
setup-node: false
@@ -112,7 +117,7 @@ jobs:
112117
id: image-name
113118

114119
open-api-ci:
115-
uses: maykinmedia/open-api-workflows/.github/workflows/ci.yml@v6
120+
uses: maykinmedia/open-api-workflows/.github/workflows/ci.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0
116121
needs:
117122
- store-reusable-workflow-vars
118123
permissions:
@@ -127,7 +132,7 @@ jobs:
127132
django-settings-module: nrc.conf.ci
128133

129134
open-api-publish:
130-
uses: maykinmedia/open-api-workflows/.github/workflows/publish.yml@v6
135+
uses: maykinmedia/open-api-workflows/.github/workflows/publish.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0
131136
needs:
132137
- store-reusable-workflow-vars
133138
- open-api-ci

.github/workflows/code_quality.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,13 +4,15 @@ name: Code quality checks
44
on:
55
push:
66
branches:
7-
- main
8-
pull_request:
7+
- main
8+
pull_request:
99
workflow_dispatch:
1010

11+
permissions: {}
12+
1113
jobs:
1214
open-api-workflow-code-quality:
13-
uses: maykinmedia/open-api-workflows/.github/workflows/code-quality.yml@v6
15+
uses: maykinmedia/open-api-workflows/.github/workflows/code-quality.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0
1416
with:
1517
python-version: '3.12'
1618
node-version: '24'

.github/workflows/codeql-analysis.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,8 @@ on:
1717
permissions:
1818
contents: read
1919
security-events: write
20+
actions: read
2021

2122
jobs:
2223
open-api-workflow-code-analysis:
23-
uses: maykinmedia/open-api-workflows/.github/workflows/code-analysis.yml@v6
24+
uses: maykinmedia/open-api-workflows/.github/workflows/code-analysis.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0

.github/workflows/oaf-check.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ permissions:
1616

1717
jobs:
1818
open-api-workflow-check-oas:
19-
uses: maykinmedia/open-api-workflows/.github/workflows/oaf-check.yml@v6
19+
uses: maykinmedia/open-api-workflows/.github/workflows/oaf-check.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0
2020

2121
with:
2222
python-version: '3.12'

.github/workflows/oas.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ permissions:
1818
jobs:
1919
oas:
2020
name: Checks
21-
uses: maykinmedia/open-api-workflows/.github/workflows/oas.yml@v6
21+
uses: maykinmedia/open-api-workflows/.github/workflows/oas.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0
2222
with:
2323
python-version: '3.12'
2424
apt-packages: 'libgdal-dev gdal-bin'

.github/workflows/quick-start.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,4 +15,4 @@ permissions:
1515

1616
jobs:
1717
open-api-workflow-quick-start:
18-
uses: maykinmedia/open-api-workflows/.github/workflows/quick-start.yml@v6
18+
uses: maykinmedia/open-api-workflows/.github/workflows/quick-start.yml@79102b911003d75203ca2fed7df01ad79d9b6bba # v6.4.0

CHANGELOG.rst

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -941,10 +941,8 @@ Features:
941941
* Tested with Open Zaak integration
942942
* Admin interface to view data created via the APIs
943943
* Scalable notification delivery workers
944-
* `NLX`_ ready (can be used with NLX)
944+
* NLX ready (can be used with NLX)
945945
* Documentation on https://open-notificaties.readthedocs.io/
946946
* Deployable on Kubernetes, single server and as VMware appliance
947947
* Automated test suite
948948
* Automated deployment
949-
950-
.. _NLX: https://nlx.io/

docs/installation/index.rst

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,12 @@ Before you begin
2121
entry. In some cases, you might want this but it's not recommended. The same machine
2222
can be used for both `Open Zaak`_ and Open Notificaties.
2323

24-
* If you want to use `NLX`_, make sure you have a publicaly available domain name, for
24+
* If you want to use NLX, make sure you have a publicaly available domain name, for
2525
example ``nlx.<organization.com>``, where your NLX-inway is accessible to the outside
2626
world.
2727

2828

2929
.. _`Open Zaak`: https://github.com/open-zaak/open-zaak
30-
.. _`NLX`: https://nlx.io/
3130

3231
Guides
3332
------

0 commit comments

Comments
 (0)