fix(cve): remove vulnerable pgproto3/v2 (CVE-2026-32286) - #1446
Conversation
khrm
left a comment
There was a problem hiding this comment.
This is also the issue here: https://github.com/tektoncd/results/security/dependabot/78
There was a problem hiding this comment.
Pull request overview
Removes the vulnerable standalone pgproto3/v2 dependency by standardizing PostgreSQL error handling on pgx/v5/pgconn.
Changes:
- Migrates remaining
pgconnv1 imports topgx/v5. - Removes obsolete dependency metadata and vendored sources.
- Updates PostgreSQL error translation tests.
Reviewed changes
Copilot reviewed 3 out of 85 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
go.mod |
Removes obsolete modules. |
go.sum |
Removes obsolete checksums. |
tools/postgres-migrate/main.go |
Uses pgx/v5/pgconn. |
pkg/api/server/db/errors/postgres/postgres_test.go |
Consolidates tests on pgx/v5. |
vendor/modules.txt |
Removes obsolete vendored modules. |
vendor/github.com/jackc/pgproto3/v2/* |
Removes vendored pgproto3/v2 package files. |
vendor/github.com/jackc/pgio/* |
Removes obsolete pgio package files. |
vendor/github.com/jackc/pgconn/* |
Removes obsolete pgconn v1 package files. |
vendor/github.com/jackc/pgconn/internal/ctxwatch/context_watcher.go |
Removes pgconn v1 internal helper. |
vendor/github.com/jackc/chunkreader/v2/* |
Removes obsolete chunkreader package files. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Migrate the two remaining usages of github.com/jackc/pgconn (v1) to github.com/jackc/pgx/v5/pgconn, eliminating the transitive dependency on github.com/jackc/pgproto3/v2 which has an unpatched DoS vulnerability (CVE-2026-32286, Dependabot tektoncd#78). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
d10c675 to
c64162d
Compare
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: divyansh42 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@khrm can you please change the release notes block to |
Migrate the two remaining usages of github.com/jackc/pgconn (v1) to github.com/jackc/pgx/v5/pgconn, eliminating the transitive dependency on github.com/jackc/pgproto3/v2 which has an unpatched DoS vulnerability (CVE-2026-32286).
Changes
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
/kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes