Skip to content

Commit 46214fb

Browse files
committed
fix: use output-based Trivy failure detection instead of exit-code
1 parent 2a1378d commit 46214fb

2 files changed

Lines changed: 8 additions & 6 deletions

File tree

.github/workflows/release-dev.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,14 @@ jobs:
3838
scan-ref: "."
3939
scanners: "vuln,secret"
4040
severity: "CRITICAL,HIGH"
41-
exit-code: "1"
41+
exit-code: "0"
4242
ignore-unfixed: true
4343
skip-dirs: ".venv,node_modules,console/node_modules,launcher,docs/site/api,console,docs/site/node_modules,docs/docusaurus/node_modules"
4444
trivyignores: ".trivyignore"
4545
format: "table"
4646
output: trivy-results.txt
4747

48-
- name: Publish Trivy results to step summary
49-
if: always()
48+
- name: Check Trivy results
5049
run: |
5150
if [[ -s trivy-results.txt ]]; then
5251
{
@@ -58,6 +57,8 @@ jobs:
5857
echo '```'
5958
echo "</details>"
6059
} >> $GITHUB_STEP_SUMMARY
60+
echo "::error::Trivy found vulnerabilities — see step summary for details"
61+
exit 1
6162
else
6263
echo "### Security Scan: No issues found" >> $GITHUB_STEP_SUMMARY
6364
fi

.github/workflows/release.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,15 +88,14 @@ jobs:
8888
scan-ref: '.'
8989
scanners: 'vuln,secret'
9090
severity: 'CRITICAL,HIGH'
91-
exit-code: '1'
91+
exit-code: '0'
9292
ignore-unfixed: true
9393
skip-dirs: '.venv,node_modules,console/node_modules,launcher,docs/site/api,console,docs/site/node_modules,docs/docusaurus/node_modules'
9494
trivyignores: '.trivyignore'
9595
format: 'table'
9696
output: trivy-results.txt
9797

98-
- name: Publish Trivy results to step summary
99-
if: always()
98+
- name: Check Trivy results
10099
run: |
101100
if [[ -s trivy-results.txt ]]; then
102101
{
@@ -108,6 +107,8 @@ jobs:
108107
echo '```'
109108
echo "</details>"
110109
} >> $GITHUB_STEP_SUMMARY
110+
echo "::error::Trivy found vulnerabilities — see step summary for details"
111+
exit 1
111112
else
112113
echo "### Security Scan: No issues found" >> $GITHUB_STEP_SUMMARY
113114
fi

0 commit comments

Comments
 (0)