Skip to content

fix(security): bump Go toolchain to 1.26.4, add govulncheck pre-commit hook#37

Merged
babs merged 1 commit into
masterfrom
fix/go-toolchain-1.26.4
Jun 4, 2026
Merged

fix(security): bump Go toolchain to 1.26.4, add govulncheck pre-commit hook#37
babs merged 1 commit into
masterfrom
fix/go-toolchain-1.26.4

Conversation

@babs

@babs babs commented Jun 4, 2026

Copy link
Copy Markdown
Owner

Why

CI govulncheck fails on every branch (including master) since publication of:

  • GO-2026-5039 — net/textproto, reachable via proxy.redirectFollowingTransport.RoundTrip
  • GO-2026-5037 — crypto/x509, reachable via the same path + mustLogger

Both fixed in go1.26.4.

What

  • go.mod: toolchain go1.26.4 (the go directive stays 1.26.3 so golangci-lint's isolated pre-commit env with GOTOOLCHAIN=local keeps loading packages; CI's setup-go reads go-version-file: go.mod and picks the toolchain up)
  • .pre-commit-config.yaml: local govulncheck ./... hook — future CVEs surface at commit time instead of in CI (requires go install golang.org/x/vuln/cmd/govulncheck@latest)

govulncheck ./... on this branch: 0 vulnerabilities affecting the code.

Unblocks #36 (will be rebased on top once this merges).

…t hook

GO-2026-5039 (net/textproto) and GO-2026-5037 (crypto/x509) are
reachable from proxy.redirectFollowingTransport / mustLogger and
fixed in go1.26.4 — CI govulncheck fails on every branch since
publication.

The go directive stays at 1.26.3 so golangci-lint's isolated
pre-commit env (GOTOOLCHAIN=local) can still load packages; the
toolchain directive upgrades actual builds and CI (setup-go reads
go-version-file: go.mod).

The new local govulncheck hook surfaces future CVEs at commit time
instead of in CI (requires golang.org/x/vuln/cmd/govulncheck).
@babs babs merged commit 3061697 into master Jun 4, 2026
7 checks passed
@babs babs deleted the fix/go-toolchain-1.26.4 branch June 4, 2026 12:43
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