Skip to content

chore(deps): resolve dependabot security vulnerabilities - #7342

Merged
msureshkumar88 merged 3 commits into
mainfrom
chore/update-dependencies-dependabot-fixes
Oct 27, 2026
Merged

msureshkumar88 merged 3 commits into
mainfrom
chore/update-dependencies-dependabot-fixes

Conversation

@prakhar-singh1928

Copy link
Copy Markdown
Collaborator

Pull Request

🔗 Related Issue

Addresses Dependabot security advisories across Python, Node.js, and Rust ecosystems.


📝 Summary

Updates Python, Node.js, and Rust dependencies to resolve open security vulnerabilities identified by Dependabot and audit tools (pip-audit, npm audit).

  • Python (pyproject.toml, uv.lock):

  • Node.js (package.json, package-lock.json):

  • Rust (Cargo.toml, crates/mcp_runtime/Cargo.toml, Cargo.lock):

    • opentelemetry_sdk: upgraded 0.31.0 -> 0.32.1
    • opentelemetry: upgraded 0.31.0 -> 0.32.0
    • opentelemetry-otlp: upgraded 0.31.1 -> 0.32.0
    • tracing-opentelemetry: upgraded 0.32.1 -> 0.33.0
    • reqwest: upgraded 0.12.28 -> 0.13.5

📏 Reviewability

  • This PR has one clear purpose
  • The linked issue is not labeled triage
  • Unrelated bugs or improvements are tracked in separate issues/PRs
  • Tests are included with the code they validate
  • If AI-assisted, I understand and can explain the generated changes

🏷️ Type of Change

  • Bug fix
  • Feature / Enhancement
  • Documentation
  • Refactor
  • Chore (deps, CI, tooling)
  • Other (describe below)

🧪 Verification

Check Command Status
Python Security Audit uv run pip-audit Passed (0 vulnerabilities)
NPM Security Audit npm audit Passed (0 vulnerabilities)
JavaScript Unit Tests npm test Passed
UI Build Validation make build-ui Passed
Python Lint Suite make ruff Passed
Rust Compilation & Tests cargo check --workspace && cargo test --workspace Passed (107 unit + 56 integration tests)
Rust Linter & Formatter cargo clippy -- -D warnings && cargo fmt -- --check Passed

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • Tests added/updated for changes
  • Documentation updated (if applicable)
  • No secrets or credentials committed

📓 Notes (optional)

All 3 package ecosystems (uv.lock, package-lock.json, and Cargo.lock) are now synchronized with zero open audit findings.

Update Python, Node.js, and Rust dependencies to fix open Dependabot security advisories.

- Update brotli, filelock, setuptools, and virtualenv in uv.lock.
- Update vitest, dompurify, retire, and undici in package-lock.json.
- Update opentelemetry_sdk, opentelemetry, and reqwest in Cargo.lock.

Signed-off-by: prakhar-singh1928 <prakhar.singh1928@ibm.com>
msureshkumar88
msureshkumar88 previously approved these changes Oct 27, 2026

@msureshkumar88 msureshkumar88 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved. The PR delivers what it claims: one dependency-only change across Python, Node.js, and Rust. It touches no application code.

Checked locally on commit ba72518:

  • cargo check --workspace --locked and cargo clippy --workspace --locked -- -D warnings pass with OpenTelemetry 0.32 and reqwest 0.13.5.
  • cargo test -p contextforge_mcp_runtime --locked passes: 107 unit tests and 2 integration tests.
  • uv lock --check and npm ci --dry-run pass. The lockfiles match the manifests.
  • test_primary_worker.py and test_bootstrap_db.py pass on filelock 4.2.4. Both create FileLock after fork, so the 3.x fork-safety guard does not apply.

Notes, none blocking:

  • Wait for CI before merge. Most jobs were still pending at review time. Watch license-check, dependency-review, and Docker Scan Complete.
  • The reqwest feature change from rustls-tls-native-roots to rustls keeps rustls-platform-verifier in the tree. The lockfile already contained it through reqwest-rmcp, so license-check should not change.
  • The filelock bump is a major version, 3.x to 4.x. The new filelock>=4.2.1 has no upper bound. Consider a ceiling, as the old pin had.

@gandhipratik203 gandhipratik203 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

Signed-off-by: prakhar-singh1928 <prakhar.singh1928@ibm.com>
Signed-off-by: prakhar-singh1928 <prakhar.singh1928@ibm.com>

@Lang-Akshay Lang-Akshay left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM ✅

@msureshkumar88
msureshkumar88 added this pull request to the merge queue Oct 27, 2026
Merged via the queue into main with commit f527977 Oct 27, 2026
75 checks passed
@msureshkumar88
msureshkumar88 deleted the chore/update-dependencies-dependabot-fixes branch October 27, 2026 13:59
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.

4 participants