Skip to content

fix: keep Trivy SARIF uploads healthy - #879

Merged
sabbour merged 10 commits into
devfrom
fix/trivy-code-scanning-config
Aug 24, 2026
Merged

sabbour merged 10 commits into
devfrom
fix/trivy-code-scanning-config

Conversation

@sabbour

@sabbour sabbour commented Aug 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • separate the Trivy SARIF upload path from the failing vulnerability gate so GitHub Security still receives results
  • refresh the AgentHost image toolchain and pinned utilities to pick up the current fixable Trivy findings
  • add a patch changeset for the maintenance/security fix

Validation

  • python YAML parse of .github/workflows/agent-host-maintenance.yml
  • git diff --check
  • verified the failing run log uploaded SARIF successfully but still failed on the gating Trivy step

Copilot AI added 2 commits August 24, 2026 13:08
Separate the Trivy SARIF upload from the vulnerability gate so GitHub Security still receives results even when the HIGH/CRITICAL gate fails, and refresh the agent host image toolchain to pick up the current fixable findings.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 11857092-1327-41bd-8708-b2ca85e674dd
The go-tools stage bumps golang.org/x/mod via 'go get' before building
buildctl from source, but buildkit vendors its dependencies. Without
running 'go mod vendor' afterward, the build fails with:
  go: inconsistent vendoring in /src/buildkit:
  golang.org/x/mod@v0.40.0: is explicitly required in go.mod, but not
  marked as explicit in vendor/modules.txt

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11857092-1327-41bd-8708-b2ca85e674dd
@sabbour
sabbour enabled auto-merge (squash) August 24, 2026 20:32
sabbour and others added 8 commits August 24, 2026 13:32
CVE-2026-56852 (golang.org/x/text DoS via invalid UTF-8) is only fixed
in x/text v0.39.0, but it's embedded in the prebuilt
@typescript/typescript-linux-x64 native tsc binary that ships with the
typescript npm package. 7.1.0-dev.20260817.1 is the newest available
prerelease and still bundles v0.38.0 — there's no newer upstream build
to bump to, so suppress this specific CVE like the other prebuilt-binary
entries this file already documents.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11857092-1327-41bd-8708-b2ca85e674dd
…tsc binary

The pinned tsc native binary (typescript 7.1.0-dev.20260817.1, latest
available prerelease) also embeds Go stdlib v1.26.4, which carries 8
more HIGH CVEs fixed only in 1.25.13/1.26.6/1.27.0-rc.3. Same
unfixable-third-party-binary situation as CVE-2026-56852 already
suppressed for this binary's x/text dependency.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 11857092-1327-41bd-8708-b2ca85e674dd
@sabbour
sabbour merged commit 28d34d3 into dev Aug 24, 2026
27 of 28 checks passed
@sabbour
sabbour deleted the fix/trivy-code-scanning-config branch August 24, 2026 23:29
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