Skip to content

all: require Go 1.25 and close the Dependabot graph - #3

Merged
tbarbugli merged 1 commit into
mainfrom
deps-govulncheck
Aug 12, 2026
Merged

all: require Go 1.25 and close the Dependabot graph#3
tbarbugli merged 1 commit into
mainfrom
deps-govulncheck

Conversation

@tbarbugli

Copy link
Copy Markdown
Member

Summary

  • Raise the module to Go 1.25 (toolchain go1.25.12) so we can take pgx 5.9.2, which is the only vulnerable module the released binary imported. That closes the two memory-safety criticals and the simple-protocol SQLi.
  • Bump testcontainers to 0.43.0, which drops github.com/docker/docker. gRPC and the old Docker CVEs leave with it.
  • Leftovers after tidy: x/crypto 0.52.0, x/text 0.39.0, otel 1.44.0.
  • CI now runs govulncheck on ./cmd/pgmigrate ./internal/... (no integration tag). Weekly grouped Dependabot for gomod minor/patch so the next bump is one PR.

govulncheck on the released surface is clean. The integration-tagged scan finds no reachable symbols; x/crypto/openpgp remains an unused, unpatched module finding.

Test plan

  • govulncheck ./cmd/pgmigrate ./internal/...
  • govulncheck -tags=integration ./... (no reachable symbols)
  • gofmt, vet, unit tests, race
  • make integration (one indexbuild container-start flake under parallel load; package retried clean)
  • make e2e (migration-e2e=ok)
  • CI green on this PR

Made with Cursor

pgx 5.9.2 is the only vulnerable module the released binary imported, and it
needs Go 1.25. testcontainers 0.43 drops github.com/docker/docker, which was
the rest of the alerts. The toolchain line is the patched stdlib; govulncheck
now runs in CI so the next one is a failing check rather than 29 open alerts.

Co-authored-by: Cursor <cursoragent@cursor.com>
@tbarbugli
tbarbugli merged commit f7652ff into main Aug 12, 2026
1 check passed
@tbarbugli
tbarbugli deleted the deps-govulncheck branch August 12, 2026 21:55
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