Skip to content

Fix urllib3 HIGH CVEs (CVE-2026-44431/44432)#528

Closed
moshemorad wants to merge 3 commits into
mainfrom
patch-urllib3-cves
Closed

Fix urllib3 HIGH CVEs (CVE-2026-44431/44432)#528
moshemorad wants to merge 3 commits into
mainfrom
patch-urllib3-cves

Conversation

@moshemorad

Copy link
Copy Markdown
Contributor

Fixes the two HIGH Dependabot alerts for urllib3 (decompression-bomb safeguards bypass in the streaming API), surfaced in Vanta.

poetry.lock and pyproject.toml already require urllib3 ^2.7.0, confirming compatibility.

Scope: HIGH severity only, as requested. MEDIUM findings (pytest, requests, idna) intentionally left out.

🤖 Generated with Claude Code

moshemorad and others added 2 commits June 8, 2026 12:29
Resolves two HIGH-severity Vanta/Dependabot findings on the krr image,
both in urllib3 2.6.3 (fixed in 2.7.0):

- CVE-2026-44431 (5.3): sensitive headers (Authorization/Cookie/
  Proxy-Authorization) forwarded on cross-origin redirects via the
  low-level ProxyManager.connection_from_url().urlopen() flow.
- CVE-2026-44432 (7.5): DoS via excessive HTTP response decompression
  (Brotli read / drain_conn) — CWE-409.

Bumps urllib3 ^2.6.3 -> ^2.7.0 in pyproject.toml, 2.6.3 -> 2.7.0 in
requirements.txt, and updates poetry.lock accordingly. Trivy rescan of
requirements.txt reports 0 HIGH/CRITICAL; full test suite passes (55).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Bump urllib3 2.6.3 -> 2.7.0 in enforcer/requirements.txt to fix the
decompression-bomb streaming-API bypass (Dependabot alert #76, HIGH).
The root requirements.txt and poetry.lock are already on 2.7.0.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9944ac4f-67a3-454b-981e-cf861b63faf6

📥 Commits

Reviewing files that changed from the base of the PR and between 2d07503 and 7ca4344.

⛔ Files ignored due to path filters (1)
  • poetry.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • enforcer/requirements.txt
  • pyproject.toml
  • requirements.txt

Walkthrough

The pull request updates the urllib3 dependency from version 2.6.3 to 2.7.0 across three configuration files: pyproject.toml, requirements.txt, and enforcer/requirements.txt. All other requirements remain unchanged.

Changes

urllib3 Version Bump

Layer / File(s) Summary
urllib3 version bump across all requirement files
pyproject.toml, requirements.txt, enforcer/requirements.txt
urllib3 is upgraded from 2.6.3 to 2.7.0 consistently across pyproject.toml (caret constraint), requirements.txt (pinned), and enforcer/requirements.txt (pinned).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • robusta-dev/krr#527: Both PRs bump the urllib3 dependency from 2.6.3 to 2.7.0 in pyproject.toml and requirements.txt with the same version constraints, representing an overlapping change.

Suggested reviewers

  • Sheeproid
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Fix urllib3 HIGH CVEs (CVE-2026-44431/44432)' accurately reflects the main objective of the PR, which updates urllib3 across multiple files to address two HIGH-severity CVEs.
Description check ✅ Passed The description is well-related to the changeset, explaining the CVEs being fixed, which requirements files are updated, and confirming compatibility with existing configurations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch patch-urllib3-cves

Comment @coderabbitai help to get the list of available commands and usage tips.

@moshemorad moshemorad closed this Jun 8, 2026
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.

1 participant