Skip to content

Commit e93805f

Browse files
fix: improve safety scan to handle vulnerabilities properly
- Capture safety exit code separately to avoid job failure - Upload safety exit code as artifact for analysis - This allows security audit to complete even with vulnerabilities found
1 parent d37e477 commit e93805f

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ jobs:
8484
pip install safety pip-audit
8585
- name: Security audit
8686
run: |
87-
safety check --json > safety-results.json || echo '{"vulnerabilities": []}' > safety-results.json
87+
safety check --json > safety-results.json; echo $? > safety-exit-code.txt
8888
pip-audit --desc --format=json --output=audit-results.json
8989
- name: Upload security audit results
9090
if: always()
@@ -93,6 +93,7 @@ jobs:
9393
name: security-audit-results
9494
path: |
9595
safety-results.json
96+
safety-exit-code.txt
9697
audit-results.json
9798
9899
pre-commit:

0 commit comments

Comments
 (0)