Skip to content

Dependencies: Raise urllib3 minimum version to 2.8.0, drop urllib3 1.x - #848

Merged
bgunebakan merged 3 commits into
mainfrom
bilal/bump-urllib3-version
Oct 29, 2026
Merged

bgunebakan merged 3 commits into
mainfrom
bilal/bump-urllib3-version

Conversation

@bgunebakan

Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

Raise the urllib3 requirement from unbounded to >=2.8.0, which removes support for urllib3 1.x.

Breaking change: users who pin urllib3<2.8 will get a resolver conflict.

Checklist

  • CHANGES.rst entry

@bgunebakan bgunebakan self-assigned this Oct 28, 2026
@bgunebakan bgunebakan added the dependencies Details concerned with dependencies. label Oct 28, 2026
@bgunebakan
bgunebakan requested a review from juanpardo October 28, 2026 10:32

@juanpardo juanpardo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I would also pin certifi because it is also part of the recent vulnerabilities

Comment thread pyproject.toml Outdated

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Please also consider pinning certifi

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I pinned the version.

@bgunebakan
bgunebakan requested a review from juanpardo October 29, 2026 08:25
@mergify

mergify Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@bgunebakan
bgunebakan merged commit fd42614 into main Oct 29, 2026
17 checks passed
@bgunebakan
bgunebakan deleted the bilal/bump-urllib3-version branch October 29, 2026 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Details concerned with dependencies.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants