[Aikido] Fix 3 critical issues in jackc/pgx/v5 and 61 other issues - #74
[Aikido] Fix 3 critical issues in jackc/pgx/v5 and 61 other issues#74aikido-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 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 112346f. Configure here.
| google.golang.org/genproto/googleapis/api v0.0.0-20250603155806-513f23925822 | ||
| google.golang.org/genproto/googleapis/rpc v0.0.0-20250603155806-513f23925822 | ||
| google.golang.org/grpc v1.73.0 | ||
| google.golang.org/protobuf v1.36.10 |
There was a problem hiding this comment.
Vendor tree left unpatched
High Severity
go.mod and go.sum bump patched libraries (for example pgx, x/crypto, kin-openapi, grpc), but vendor/ still pins the old vulnerable versions. The main Dockerfile builds with GOFLAGS=-mod=vendor, so Docker images keep shipping the unpatched deps this PR claims to fix.
Reviewed by Cursor Bugbot for commit 112346f. 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 pre-release pinned
High Severity
google.golang.org/grpc is pinned to v1.81.0-dev, a master development tag, while stable v1.81.0 already exists. The -dev tag is far behind the release and omits later fixes, so production gets an unfinished, moving pre-release instead of the intended patched gRPC.
Reviewed by Cursor Bugbot for commit 112346f. Configure here.


Description
Upgrade dependencies to fix critical SSH vulnerabilities including authorization bypass, resource exhaustion DoS, FIDO key verification bypass, integer overflow, and CA key revocation bypass.
Motivation
Security Impact — CVE vulnerabilities fixed by this PR:
✅ 64 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.param[items][50000000]=x.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
Wide dependency upgrades touch Postgres (pgx), gRPC authorization, OpenAPI validation, SSH/crypto, and Docker/archive code paths with no accompanying code changes—regression and the grpc 1.81.0-dev pin warrant careful CI and integration testing.
Overview
This PR is a dependency-only update:
go.modandgo.sumchange with no application source edits.The module now targets Go 1.25.0 (from 1.24.0) and drops the explicit
toolchainpin. Direct and transitive dependencies are refreshed to address a large set of reported CVEs, especially in networking, crypto, and data-access stacks.Notable direct bumps include
github.com/jackc/pgx/v5(5.5.4 → 5.9.2),golang.org/x/crypto,golang.org/x/net,google.golang.org/grpc(1.73.0 → 1.81.0-dev),github.com/getkin/kin-openapi(0.132.0 → 0.144.0),github.com/modelcontextprotocol/go-sdk,go.opentelemetry.io/otel(and related sdk/metric/trace packages to 1.43.0),github.com/golang-jwt/jwt/v5, andgithub.com/docker/docker/github.com/moby/go-archivevia testcontainers. Several OpenAPI-related indirect dependencies also shift (e.g. removal of somego-openapi/swag/mailru/easyjsonpaths, newsanthosh-tekuri/jsonschemaandsegmentio/encodingentries).Reviewers should treat this as a broad supply-chain security refresh with regression risk from the gRPC pre-release bump and major version jumps in libraries used for Postgres (
pgx), OpenAPI tooling, MCP, and container/archive handling in tests.Reviewed by Cursor Bugbot for commit 112346f. Bugbot is set up for automated code reviews on this repo. Configure here.