[Aikido] Fix 3 critical issues in jackc/pgx/v5 and 59 other issues - #69
[Aikido] Fix 3 critical issues in jackc/pgx/v5 and 59 other issues#69aikido-autofix[bot] wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit cb7f762. Configure here.
| google.golang.org/genproto/googleapis/api v0.0.0-20260414002931-afd174a4e478 | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20260414002931-afd174a4e478 | ||
| google.golang.org/grpc v1.81.0-dev | ||
| google.golang.org/protobuf v1.36.11 |
There was a problem hiding this comment.
Vendor tree left stale
High Severity
go.mod bumps critical modules such as jackc/pgx/v5, golang.org/x/crypto, golang.org/x/net, and google.golang.org/grpc, but vendor/ still pins the old vulnerable versions. Dockerfile builds with GOFLAGS=-mod=vendor, so image builds hit inconsistent vendoring and would otherwise keep shipping the unfixed deps this PR claims to remediate.
Reviewed by Cursor Bugbot for commit cb7f762. Configure here.
| gonum.org/v1/gonum v0.17.0 | ||
| google.golang.org/genproto/googleapis/api v0.0.0-20260414002931-afd174a4e478 | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20260414002931-afd174a4e478 | ||
| google.golang.org/grpc v1.81.0-dev |
There was a problem hiding this comment.
Unstable gRPC dependency pin
High Severity
google.golang.org/grpc is pinned to v1.81.0-dev, a pre-release tag, even though stable v1.81.0 / v1.81.1 exist. That leaves the security upgrade on a non-release line that is not reproducible and misses later stable patches such as the v1.81.1 xDS RBAC fix.
Reviewed by Cursor Bugbot for commit cb7f762. Configure here.
| go 1.24.0 | ||
|
|
||
| toolchain go1.24.4 | ||
| go 1.25.0 |
There was a problem hiding this comment.
Go version pipeline mismatch
High Severity
go.mod now requires go 1.25.0, but Dockerfile still builds on golang:1.24-alpine and the release workflow still installs Go 1.24. Those paths fail (or silently diverge) because the module language version exceeds the pinned toolchains, especially when GOTOOLCHAIN=local in official images.
Reviewed by Cursor Bugbot for commit cb7f762. Configure here.
|
Closed by Aikido: a new AutoFix has been created → #74 |


Description
Upgrade dependencies to fix critical SSH vulnerabilities including authorization bypass, resource exhaustion DoS, FIDO key verification bypass, integer overflow, and certificate revocation bypass.
Motivation
Security Impact — CVE vulnerabilities fixed by this PR:
✅ 62 CVEs resolved by this upgrade, including 17 critical 🚨 CVEs
This PR will resolve the following CVEs:
:pathvalidation allows requests without leading slashes to bypass path-based authorization interceptors, enabling attackers to circumvent "deny" rules and access restricted gRPC methods. This authorization bypass affects servers using path-based RBAC policies with fallback "allow" rules.ReadFromFilefunction due to insufficient input sanitization, allowing attackers to access files outside the intended directory.Type of change (choose one)
Checklist
Check our Pull Request Guidelines
Note
High Risk
Touches Postgres (pgx), JWT, gRPC, crypto, and OpenAPI stacks used for data and auth. grpc is pinned to the pre-release
v1.81.0-dev, and the Go 1.25 bump can break CI/tooling.Overview
Security-driven
go.mod/go.sumrefresh only: Go 1.25.0 (drops thetoolchain go1.24.4pin) plus upgrades meant to close CVEs in Postgres, crypto, networking, JWT, OpenAPI, MCP, and OpenTelemetry.Notable direct bumps:
jackc/pgx/v55.5.4 → 5.9.2,golang.org/x/crypto0.45 → 0.53,golang.org/x/net0.47 → 0.56,golang-jwt/jwt/v55.2.2 → 5.3.0,getkin/kin-openapi0.132 → 0.144,modelcontextprotocol/go-sdk1.0.0 → 1.4.1, and OTel 1.37 → 1.43.google.golang.org/grpcmoves from 1.73.0 tov1.81.0-dev.github.com/docker/dockerstays atv28.3.3+incompatibledespite related findings in the PR write-up. No source changes beyond the module files.Reviewed by Cursor Bugbot for commit cb7f762. Bugbot is set up for automated code reviews on this repo. Configure here.