Skip to content

🔒 build(go): bump go directive to 1.26.6 so CI ships a patched stdlib - #195

Open
konih wants to merge 1 commit into
mainfrom
fix-go-toolchain-1.26.6
Open

🔒 build(go): bump go directive to 1.26.6 so CI ships a patched stdlib#195
konih wants to merge 1 commit into
mainfrom
fix-go-toolchain-1.26.6

Conversation

@konih

@konih konih commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • task vuln:check — 0 vulnerabilities (was 6)
  • task test:run — all packages pass, internal/ coverage 92.7% (floor 90%), api/ coverage 92.5% (floor 75%)

Taskfile.yml derives GOTOOLCHAIN straight from the `go` directive line
(sed 's/^go //p'), so a separate `toolchain` line (as used in assent) is
ignored here. govulncheck flagged 6 stdlib CVEs (GO-2026-6218/6091/6090/
6089/5972/5026) fixed in go1.26.6; main's Vulncheck workflow has been red
since 2026-08-17 at 6ce4ec4, independent of and blocking the open
Dependabot PRs (#193, #194).
@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

Copy link
Copy Markdown

@konih

konih commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator Author

Blocked: this go.mod bump (1.26.5 → 1.26.6) fixes `test`/`vulncheck` but breaks `docker-build` and `e2e (kustomize)`, since `Dockerfile` pins the builder stage to `golang:1.26.5@sha256:...` — now go mod download fails with `go.mod requires go >= 1.26.6 (running go 1.26.5; GOTOOLCHAIN=local)`.

Needs a coordinated bump of the Dockerfile's builder FROM line to `golang:1.26.6` with the correct digest. Running this from an unattended sandboxed sweep with no Docker Hub / WebFetch network access — could not resolve the digest safely (didn't want to drop the digest pin as a workaround, that's a security regression). Left for operator: bump `Dockerfile` line 2 to `golang:1.26.6@sha256:`, then this PR (plus the two blocked Dependabot PRs #193/#194) can go green.

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