Skip to content

Fix Snyk security workflow: missing SARIF output, deprecated action, unguarded upload - #73

Draft
Mapuppy09 with Copilot wants to merge 2 commits into
latestfrom
copilot/fix-failing-github-actions-job
Draft

Fix Snyk security workflow: missing SARIF output, deprecated action, unguarded upload#73
Mapuppy09 with Copilot wants to merge 2 commits into
latestfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown

The Snyk security workflow was failing with Path does not exist: snyk.sarif because the scan step never wrote the file, and the upload step ran unconditionally against a nonexistent path using a deprecated action version.

Changes

  • Generate SARIF file: Added --sarif-file-output=snyk.sarif to Snyk scan args — without this the file is never created
  • Guard upload step: Added hashFiles('snyk.sarif') != '' to the upload if: condition so it skips cleanly when Snyk fails to produce output
  • Non-blocking scan: Added continue-on-error: true to the Snyk step so vulnerability findings (exit 1) don't prevent the upload step from running
  • Upgrade deprecated action: github/codeql-action/upload-sarif@v2@v3 (v2 was deprecated January 2025)

@snyk-io

snyk-io Bot commented Aug 27, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

…d upload

Co-authored-by: Mapuppy09 <144494139+Mapuppy09@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix the failing GitHub Actions job Fix Snyk security workflow: missing SARIF output, deprecated action, unguarded upload Aug 27, 2026
Copilot AI requested a review from Mapuppy09 August 27, 2026 16:51
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.

2 participants