Skip to content

fix: resolve all 24 open Dependabot security alerts#1179

Merged
jchrostek-dd merged 1 commit into
mainfrom
john/dependabot-review
Apr 15, 2026
Merged

fix: resolve all 24 open Dependabot security alerts#1179
jchrostek-dd merged 1 commit into
mainfrom
john/dependabot-review

Conversation

@jchrostek-dd

Copy link
Copy Markdown
Contributor

Summary

Resolves all 24 open Dependabot security alerts (2 critical, 6 high, 14 medium, 2 low) across three ecosystems by updating vulnerable dependencies in test infrastructure.

Note: All affected dependencies are in test infrastructure (integration-tests/, local_tests/), not the production Rust extension (bottlecap/).

Changes

npm (integration-tests/)

  • axios ^1.6.7^1.15.0 — fixes SSRF bypass via NO_PROXY hostname normalization (CVE-2025-62718, critical)

Go (local_tests/golang/)

  • golang.org/x/net v0.2.0v0.52.0 — fixes 8 alerts: HTTP/2 rapid reset, uncontrolled resource consumption, XSS, IPv6 zone ID proxy bypass, excessive memory growth, stream cancellation, improper text rendering
  • google.golang.org/grpc v1.50.1v1.80.0 — fixes 2 alerts: authorization bypass via missing leading slash (CVE-2026-33186, critical), HTTP/2 rapid reset
  • google.golang.org/protobuf v1.28.1v1.36.11 — fixes infinite loop in protojson.Unmarshal
  • github.com/golang/glog v1.0.0v1.2.5 — fixes insecure temporary file usage

Python (local_tests/serverless-init/)

  • Flask 2.1.23.1.3 — fixes session cookie disclosure, missing Vary:Cookie header
  • Werkzeug 2.2.23.1.8 — fixes 9 alerts: debugger RCE, safe_join Windows device name bypasses, multipart form DoS, resource exhaustion, nameless cookie bypass

Reviewer Notes

⚠️ Go minimum version: The go directive in local_tests/golang/go.mod was auto-upgraded from go 1.22 to go 1.25.0 (required by golang.org/x/net v0.52.0). This only affects the test Lambda function, not CI build toolchains for the main extension.

⚠️ Flask/Werkzeug major version jump: Flask 2.x → 3.x and Werkzeug 2.x → 3.x are major version bumps. The test app (app.py) uses minimal Flask APIs and was reviewed for compatibility. However, ddtrace==1.4.1 compatibility with Flask 3.x should be verified in CI.

Alert Breakdown

Severity Count Alerts
Critical 2 #128 (axios SSRF), #104 (gRPC auth bypass)
High 6 #52, #43, #40, #38, #37, #20
Medium 14 #88, #74, #73, #70, #68, #65, #60, #59, #53, #51, #49, #45, #41, #36
Low 2 #89, #39

Update dependencies across npm, Go, and Python ecosystems to address
all open Dependabot security alerts (2 critical, 6 high, 14 medium, 2 low).

npm (1 alert):
- axios: ^1.6.7 -> ^1.15.0 (fixes CVE-2025-62718 SSRF bypass)

Go (12 alerts):
- golang.org/x/net: v0.2.0 -> v0.52.0 (fixes 8 alerts including HTTP/2 vulns)
- google.golang.org/grpc: v1.50.1 -> v1.80.0 (fixes 2 alerts including auth bypass)
- google.golang.org/protobuf: v1.28.1 -> v1.36.11 (fixes infinite loop CVE)
- github.com/golang/glog: v1.0.0 -> v1.2.5 (fixes insecure temp file usage)

Python (11 alerts):
- Werkzeug: 2.2.2 -> 3.1.8 (fixes 9 alerts including RCE via debugger)
- Flask: 2.1.2 -> 3.1.3 (fixes 2 alerts including session cookie disclosure)

All affected manifests are in test infrastructure (integration-tests/,
local_tests/), not the production Rust extension (bottlecap/).

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

@lym953 lym953 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@jchrostek-dd
jchrostek-dd marked this pull request as ready for review April 15, 2026 16:41
@jchrostek-dd
jchrostek-dd requested a review from a team as a code owner April 15, 2026 16:41
@jchrostek-dd
jchrostek-dd merged commit eb8db5f into main Apr 15, 2026
55 checks passed
@jchrostek-dd
jchrostek-dd deleted the john/dependabot-review branch April 15, 2026 16:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates vulnerable dependencies used by this repository’s test infrastructure (integration tests + local test apps) to clear Dependabot security alerts, without modifying the production Rust extension in bottlecap/.

Changes:

  • Bump axios in integration-tests/ to ^1.15.0 (and update lockfile accordingly).
  • Update Go module dependencies in local_tests/golang/, including golang.org/x/net, grpc, and protobuf.
  • Update Python test app dependencies in local_tests/serverless-init/ to Flask 3.x / Werkzeug 3.x.

Reviewed changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
local_tests/serverless-init/requirements.txt Updates Flask/Werkzeug versions for security fixes in the serverless-init local test app.
local_tests/golang/go.mod Updates Go dependency versions and bumps the Go toolchain directive for the Go local test app.
local_tests/golang/go.sum Synchronizes Go module checksums for updated Go dependencies.
integration-tests/package.json Updates axios version range for integration test tooling.
integration-tests/package-lock.json Locks axios (and transitive deps) to the updated versions.
Files not reviewed (1)
  • integration-tests/package-lock.json: Language not supported

Comment thread local_tests/golang/go.mod
module app

go 1.22
go 1.25.0

Copilot AI Apr 15, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The go directive was bumped from 1.22 to 1.25.0, which increases the minimum Go toolchain required to build/run this local test app. If this version bump isn’t strictly required by the updated deps, consider keeping the directive at the lowest supported version (or at least using the repo/CI’s Go version) to avoid breaking contributors’ local test workflows; if it is required, it would help to document the new minimum Go version for local_tests/golang/.

Suggested change
go 1.25.0
go 1.22

Copilot uses AI. Check for mistakes.
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.

3 participants