aboutsummaryrefslogtreecommitdiffstats
path: root/.github
diff options
context:
space:
mode:
authorJoas Schilling <coding@schilljs.com>2024-10-01 00:09:15 +0200
committerJoas Schilling <coding@schilljs.com>2024-10-01 00:11:07 +0200
commit570a9e208fa638c35f5ec1c9754acf2a66aa587d (patch)
treef1242f593b5afe10596cc7b599a3e444fd056f2c /.github
parent990ee44015da720acbcf22dfe583b181dd1655fd (diff)
downloadnextcloud-server-570a9e208fa638c35f5ec1c9754acf2a66aa587d.tar.gz
nextcloud-server-570a9e208fa638c35f5ec1c9754acf2a66aa587d.zip
ci: Add psalm baseline for security and make CI fail on changeci/noid/improve-running-psalm-locally
Signed-off-by: Joas Schilling <coding@schilljs.com>
Diffstat (limited to '.github')
-rw-r--r--.github/CODEOWNERS2
-rw-r--r--.github/workflows/static-code-analysis.yml6
2 files changed, 6 insertions, 2 deletions
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
index e448bf922ce..afe17a95f84 100644
--- a/.github/CODEOWNERS
+++ b/.github/CODEOWNERS
@@ -37,9 +37,9 @@
/apps/files_trashbin/src* @skjnldsv
# Security team
+/build/psalm-baseline-security.xml @nickvergessen
/resources/codesigning @mgallien @miaulalala @nickvergessen
/resources/config/ca-bundle.crt @ChristophWurst @miaulalala @nickvergessen
-/.drone.yml @nickvergessen
# Two-Factor Authentication
# https://github.com/nextcloud/wg-two-factor-authentication#members
diff --git a/.github/workflows/static-code-analysis.yml b/.github/workflows/static-code-analysis.yml
index 63ad51a26c6..c5e8e957077 100644
--- a/.github/workflows/static-code-analysis.yml
+++ b/.github/workflows/static-code-analysis.yml
@@ -60,7 +60,11 @@ jobs:
run: composer i
- name: Psalm taint analysis
- run: composer run psalm:security -- --threads=1 --monochrome --no-progress --output-format=github --report=results.sarif
+ run: composer run psalm:security -- --threads=1 --monochrome --no-progress --output-format=github --update-baseline --report=results.sarif
+
+ - name: Show potential changes in Psalm baseline
+ if: always()
+ run: git diff --exit-code -- . ':!lib/composer'
- name: Upload Security Analysis results to GitHub
if: always()