Skip to content

fix: bump Go from 1.25.9 to 1.25.10 to resolve Snyk vulnerabilities#869

Merged
dangrondahl merged 1 commit into
mainfrom
fix-go-version-snyk
May 8, 2026
Merged

fix: bump Go from 1.25.9 to 1.25.10 to resolve Snyk vulnerabilities#869
dangrondahl merged 1 commit into
mainfrom
fix-go-version-snyk

Conversation

@dangrondahl
Copy link
Copy Markdown
Contributor

@dangrondahl dangrondahl commented May 8, 2026

Summary

  • Bump Go version in go.mod from 1.25.9 to 1.25.10
  • Fixes 6 CVEs in Go stdlib (std/html/template, std/net, std/net/http) that cause the Snyk Dependency Test CI job to fail

CVEs resolved: CVE-2026-27142, CVE-2026-33811, CVE-2026-33814, CVE-2026-39823, CVE-2026-39826, CVE-2026-39836

Test plan

  • Snyk Dependency Test CI job passes
  • All other CI jobs pass (build, lint, tests)

Fixes CVEs in Go stdlib (std/html/template, std/net, std/net/http):
CVE-2026-27142, CVE-2026-33811, CVE-2026-33814,
CVE-2026-39823, CVE-2026-39826, CVE-2026-39836
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 8, 2026

Claude finished @dangrondahl's task in 3m 6s —— View job


Review of PR #869

  • Run static analysis (go build, go vet, go mod tidy)
  • Review the diff
  • Post review feedback

Static Analysis

All checks pass:

  • go build ./... — compiles cleanly
  • go vet ./... — no issues
  • go mod tidygo.mod and go.sum are already clean (no diff)

Code Review

Diff: Single line change in go.modgo 1.25.9go 1.25.10.

This is a minimal, well-scoped patch version bump. Nothing else to flag:

  • No edge cases or bugs — this is a toolchain directive, not application logic.
  • No security concerns — the change resolves 6 CVEs, which is the right move.
  • No test impact — the Go version bump doesn't change any application behavior, only patches stdlib vulnerabilities.
  • go.sum is unchanged — confirmed via go mod tidy && git diff --exit-code, meaning no dependency graph changes.

Good Work

Clean, minimal PR — one line, clear commit message, CVEs listed in the description. This is exactly how a version-bump PR should look.

Verdict: LGTM. Ship it once CI is green.

@dangrondahl dangrondahl enabled auto-merge (squash) May 8, 2026 11:01
@dangrondahl dangrondahl merged commit 1c3c462 into main May 8, 2026
16 checks passed
@dangrondahl dangrondahl deleted the fix-go-version-snyk branch May 8, 2026 11:03
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