Skip to content

fix(deps): bump filelock, msgpack, setuptools to clear pip-audit CVEs - #832

Open
rajnisht7 wants to merge 1 commit into
agentrust-io:mainfrom
rajnisht7:fix-dep-cve
Open

rajnisht7 wants to merge 1 commit into
agentrust-io:mainfrom
rajnisht7:fix-dep-cve

Conversation

@rajnisht7

Copy link
Copy Markdown
Member

What

Bumped filelock, msgpack, and setuptools in requirements/dev.in / requirements/dev.txt to versions that fix 3 CVEs flagged by pip-audit in CI.

Why

CI security scan failing on main, pip-audit found 5 known vulnerabilities across these 3 packages (filelock 3.32.5, msgpack 1.2.2, setuptools 84.0.0).

Security impact

None

Test plan

  • pytest passes
  • ruff check passes
  • mypy passes
  • Manual test performed (describe steps below if applicable)

executed the full CI security-scan step locally in a fresh env after installing the patched dev.txt with --require-hashes: bandit -r src/ -c pyproject.toml && pip-audit --skip-editable now exits clean with “No known vulnerabilities found”. Also recompiled runtime.txt against the updated constraints (no changes needed there, since those packages aren’t in its tree).

DCO sign-off

Signed-off-by: rajnisht7 <rajnishtiwari9787@gmail.com>
@rajnisht7
rajnisht7 enabled auto-merge (squash) October 28, 2026 01:18
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@dinakarjs dinakarjs 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.

Source/CI review at 5f25e11: the diff changes the development input constraints and generated lock for filelock, msgpack and setuptools; all 15 returned hosted checks are successful.

This overlaps my open #822, which covers filelock/setuptools. Please select one integration route rather than landing both independently: if #832 is accepted, #822 can be marked superseded after confirming its intended dependency repairs are included. #832 also covers msgpack and adds explicit input floors, so the two PRs are not identical.

I inspected the diff and hosted results; I did not independently reinstall this lock, verify artifact hashes or rerun the suite in this pass. This is a COMMENT, not independent approval of the overlapping work. Reviewed with OpenAI Codex.

This branch has not been deployed

No deployments
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