Skip to content

fix: add SARIF output and artifact upload to trivy pre-publish scan#41

Merged
tehw0lf merged 1 commit intomainfrom
feature/trivy-sarif-pre-publish-scan
Mar 14, 2026
Merged

fix: add SARIF output and artifact upload to trivy pre-publish scan#41
tehw0lf merged 1 commit intomainfrom
feature/trivy-sarif-pre-publish-scan

Conversation

@tehw0lf
Copy link
Owner

@tehw0lf tehw0lf commented Mar 14, 2026

Summary

  • Add security-events: write permission required for SARIF upload
  • Switch trivy pre-publish scan from --format table to --format sarif with file output
  • Add table output to step summary for human-readable results
  • Upload SARIF to GitHub Security tab (category: trivy-pre-publish)
  • Upload .sarif as downloadable artifact per matrix image

Test plan

  • Verify trivy scan step generates .sarif file
  • Verify SARIF appears in GitHub Security tab under trivy-pre-publish
  • Verify artifact is downloadable from workflow run
  • Verify scan still fails on findings (exit-code 1 preserved)

- Add security-events: write permission for SARIF upload
- Output SARIF file alongside table summary in step summary
- Upload SARIF to GitHub Security tab (category: trivy-pre-publish)
- Upload .sarif as downloadable artifact per matrix image
@tehw0lf tehw0lf merged commit 7111326 into main Mar 14, 2026
2 checks passed
@tehw0lf tehw0lf deleted the feature/trivy-sarif-pre-publish-scan branch March 14, 2026 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant