Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

backport HTTPConnection._tunnel from 3.11.4 to unignore remaining ResourceWarnings #3252

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

graingert
Copy link
Contributor

@graingert graingert commented Dec 31, 2023

This PR backports the fix from python/cpython#103472 in 3.11.4 so that the last ignored ResourceWarnings are no longer issued by urllib3

@graingert graingert changed the title backport HTTPConnection._tunnel from 3.11.4 backport HTTPConnection._tunnel from 3.11.4 to unignore remaining ResourceWarnings Jan 3, 2024
@graingert graingert marked this pull request as ready for review January 7, 2024 18:05
@sethmlarson
Copy link
Member

@graingert I would love to avoid bundling the _tunnel() implementation if possible, instead what do you think about a conditional filterwarnings for Python 3.11.3 or earlier so we're ensuring that all future versions continue to not emit those warnings?

@graingert
Copy link
Contributor Author

@sethmlarson but this is a warning emitted for users of urllib3 on 3.10 and I'd like the warning fixed for those people too

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.

None yet

2 participants