Skip to content

Fix/security fixes#105

Merged
glageju merged 1 commit into
mainfrom
fix/security-fixes
May 12, 2026
Merged

Fix/security fixes#105
glageju merged 1 commit into
mainfrom
fix/security-fixes

Conversation

@glageju
Copy link
Copy Markdown
Contributor

@glageju glageju commented May 12, 2026

Summary:

Grype image scans against main started failing on the medium severity cutoff. The Ko-built binary embeds the Go stdlib version, and Go 1.26.2 (current main) is missing 21 stdlib CVE fixes — including 1 Critical (CVE-2026-27143)
and 12 High — that landed in 1.26.3.

Change:

  • go.mod: go 1.26.2 → go 1.26.3. CI workflows resolve Go via go-version-file: 'go.mod', so no workflow edits needed.

Verification:

  • Rebuilt the Ko image locally with Go 1.26.3 → grype --only-fixed --fail-on medium: No vulnerabilities found
  • Filesystem scan on a clean checkout simulation: No vulnerabilities found

@glageju glageju force-pushed the fix/security-fixes branch from ca1d41d to fd76e3e Compare May 12, 2026 04:42
@glageju glageju requested a review from ChrisJBurns May 12, 2026 04:44
@glageju glageju marked this pull request as ready for review May 12, 2026 14:38
@glageju glageju requested review from lorr1 and tgrunnagle May 12, 2026 14:38
@glageju glageju merged commit 455ba14 into main May 12, 2026
5 checks passed
@rdimitrov rdimitrov mentioned this pull request May 12, 2026
2 tasks
rdimitrov added a commit that referenced this pull request May 12, 2026
Use `go 1.26` instead of `go 1.26.3` to match toolhive, toolhive-catalog,
and toolhive-core. CI uses `go-version-file: 'go.mod'`, so setup-go will
resolve to the latest available 1.26.x and keep picking up stdlib CVE
fixes automatically — which is what the 1.26.3 pin in #105 was after.

`go mod tidy` also dropped some stale go.sum entries left over from
previous bumps.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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