Skip to content

fix(deps): bump x/net + x/text to clear the image Trivy scan#150

Merged
heymrbox merged 1 commit into
mainfrom
ci-trivy-go-bump
Jul 17, 2026
Merged

fix(deps): bump x/net + x/text to clear the image Trivy scan#150
heymrbox merged 1 commit into
mainfrom
ci-trivy-go-bump

Conversation

@EdHasNoLife

Copy link
Copy Markdown
Collaborator

The Image Build job started failing on 2026-07-15 — not caused by any PR. Two newly-published CVEs landed in Trivy's vuln DB; the same image bytes scanned clean on 07-14 (count 0), then went 0→1→2 across runs on 07-15 (#134, #139, #148).

CVE Package Bump Issue
CVE-2026-46600 golang.org/x/net v0.55.0 → v0.56.0 panic parsing an invalid SVCB/HTTPS RR in dns/dnsmessage
CVE-2026-56852 golang.org/x/text v0.37.0 → v0.39.0 infinite loop in norm.Iter on invalid UTF-8

Both are indirect and not called by our code — which is exactly why govulncheck (call-based) stayed green while Trivy (version-based, whole-image) flagged them. So the fix is a dependency bump, not a code change. x/sync/x/sys/x/term move transitively.

Verified locally: go version -m bin/{controller,server} now embeds x/net v0.56.0 + x/text v0.39.0 (the build info Trivy reads); make ci green.

Unblocks the image scan for #148 and every subsequent PR/main run.

The Image Build job started failing on 2026-07-15 when two newly-published CVEs
landed in Trivy's vuln DB. Not caused by any change: the same image bytes scanned
clean on 07-14 (count 0), then 0->1->2 across runs on 07-15.

- CVE-2026-46600  golang.org/x/net  v0.55.0 -> v0.56.0
  (panic parsing an invalid SVCB/HTTPS RR in dns/dnsmessage)
- CVE-2026-56852  golang.org/x/text v0.37.0 -> v0.39.0
  (infinite loop in norm.Iter on invalid UTF-8)

Both are indirect and not called by our code, which is why govulncheck stayed green
while Trivy (version-based, whole-image) flagged them — so the fix is a dependency
bump, not a code change. x/sync, x/sys and x/term move transitively.

Verified: go version -m bin/{controller,server} now embeds x/net v0.56.0 and
x/text v0.39.0 (the build info Trivy reads); make ci green.
@github-actions

Copy link
Copy Markdown

Codex review

Files reviewed

Repository root:

  • go.mod [reviewed]
  • go.sum [reviewed]

Findings

Blocking

None.

Should-fix

None.

Nit

None.

Per-category coverage

  1. Vendor-neutral naming: Clean
  2. Contract & spec / docs sync: Clean
  3. gRPC fail-open semantics: Clean
  4. Quality (tests / error wrapping / package placement / generated drift): Clean

Verdict

approve

@heymrbox
heymrbox merged commit 20357db into main Jul 17, 2026
14 checks passed
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