Skip to content

fix(deps): bump golang.org/x/net to v0.60.0 and Go to 1.26.9 - #708

Merged
timflannagan merged 1 commit into
mainfrom
ilackarms/cve-go-6612-oss
Oct 16, 2026
Merged

timflannagan merged 1 commit into
mainfrom
ilackarms/cve-go-6612-oss

Conversation

@ilackarms

Copy link
Copy Markdown
Contributor

Description

Motivation: the server image is flagged CRITICAL for GO-2026-6612 (CVE-2026-78663), an HTTP/2 flow-control issue in golang.org/x/net/http2 and the copy bundled in net/http. The fix ships in golang.org/x/net v0.60.0 and Go 1.26.9.

What changed (go.mod/go.sum only):

  • golang.org/x/net v0.56.0 -> v0.60.0. MVS raises the rest of golang.org/x with it: crypto v0.57.0, mod v0.41.0, sync v0.23.0, sys v0.48.0, term v0.46.0, text v0.42.0.
  • go directive 1.26.4 -> 1.26.9. CI and the release job install Go from go-version-file: go.mod, so without this the arctl release binaries would still build on go1.26.4. It also guards the server image: the golang:1.26-alpine builder runs with GOTOOLCHAIN=local, so a stale cached base older than 1.26.9 now fails with go.mod requires go >= 1.26.9 instead of building a vulnerable stdlib.

Change Type

/kind fix
/kind bump

Changelog

Bump golang.org/x/net to v0.60.0 and Go to 1.26.9 to fix GO-2026-6612 (CVE-2026-78663).

Additional Notes

Scanned a local linux/arm64 build of docker/server.Dockerfile with grype (--only-fixed) and trivy (--severity HIGH,CRITICAL):

Image grype Critical / High trivy Critical / High
main @ c234049 1 / 7 0 / 3
this PR 0 / 1 0 / 1
  • Cleared: GO-2026-6612, GO-2026-6603, GO-2026-6610, GO-2026-6611, CVE-2026-78669 (x/net), plus the x/crypto v0.53.0 and x/text v0.39.0 HIGHs picked up by the same bump.
  • The main build was already on go1.26.9 because golang:1.26-alpine now resolves there; the published v0.4.0 image also carries the stdlib findings (go1.26.4/go1.26.5).
  • go version -m on the new binary: go1.26.9, golang.org/x/net v0.60.0.
  • trivy rates CVE-2026-78663 MEDIUM, which is why it shows no Critical; after this change it reports no x/net or stdlib findings at any severity.
  • The one remaining HIGH is CVE-2026-84782 in libssl3 from the ubuntu:22.04 runtime base, unrelated to this change.

Local checks: go build ./..., make test-unit (1551 passed), make verify clean.

Fixes GO-2026-6612 (CVE-2026-78663), an HTTP/2 flow-control issue in
golang.org/x/net/http2 and the bundled copy in net/http.

- golang.org/x/net v0.56.0 -> v0.60.0, which also clears GO-2026-6603,
  GO-2026-6610, GO-2026-6611 and CVE-2026-78669. MVS raises the rest of
  golang.org/x with it (crypto v0.57.0, mod v0.41.0, sync v0.23.0,
  sys v0.48.0, term v0.46.0, text v0.42.0), which also clears the
  x/crypto and x/text HIGH findings on the current image.
- go directive 1.26.4 -> 1.26.9. CI and the release job install Go from
  go.mod, so this pins those builds (including the arctl release
  binaries) to the patched stdlib. The server image's golang:1.26-alpine
  builder runs with GOTOOLCHAIN=local, so a stale cached base older than
  1.26.9 now fails the build instead of shipping a vulnerable stdlib.
Copilot AI balanced review requested due to automatic review settings October 14, 2026 18:59

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

No blocking issues were identified in the dependency-only changes, and the PR reports successful build, test, and verification checks.

0 open findings

What changed in this PR

Updates Go and related dependencies to address HTTP/2 security vulnerabilities in the registry and CLI builds.

Changes:

  • Raises the minimum Go version to 1.26.9.
  • Upgrades golang.org/x/net to v0.60.0 and related modules.
  • Refreshes dependency checksums.
File Description
go.sum Updates checksums for upgraded dependencies.
go.mod Raises the Go minimum and dependency versions.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@timflannagan
timflannagan added this pull request to the merge queue Oct 16, 2026
Merged via the queue into main with commit 0e86966 Oct 16, 2026
7 checks passed
@timflannagan
timflannagan deleted the ilackarms/cve-go-6612-oss branch October 16, 2026 00:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants