Skip to content

Remove strict kwarg that was deprecated in Python 3.4#3036

Merged
nateprewitt merged 1 commit intoboto:developfrom
nateprewitt:strict-less
Oct 10, 2023
Merged

Remove strict kwarg that was deprecated in Python 3.4#3036
nateprewitt merged 1 commit intoboto:developfrom
nateprewitt:strict-less

Conversation

@nateprewitt
Copy link
Copy Markdown
Contributor

This PR will stop passing the no-longer required strict argument to the urllib3 ConnectionPool class. This argument was originally needed in Python<3.5 (specifically Python 2) to specify support for older HTTP status lines before http/1.0. The functionality went away in HTTPConnection in Python 3.4 (ref).

Starting in urllib3 2.1, this parameter will be removed from the APIs we're using. Given it has no effect on version of Python we support, this should be a no-op for users on all versions of urllib3 we support. You can find a similar PR where we've done this in Requests which has been in place for ~6 months without issue.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 9, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (9b401c7) 93.43% compared to head (3dddb5e) 93.46%.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3036      +/-   ##
===========================================
+ Coverage    93.43%   93.46%   +0.02%     
===========================================
  Files           66       66              
  Lines        13963    13963              
===========================================
+ Hits         13047    13050       +3     
+ Misses         916      913       -3     
Files Coverage Δ
botocore/httpsession.py 92.77% <ø> (ø)

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@nateprewitt nateprewitt merged commit f0ff061 into boto:develop Oct 10, 2023
@nateprewitt nateprewitt deleted the strict-less branch October 10, 2023 20:49
@GavinZZ
Copy link
Copy Markdown

GavinZZ commented Oct 11, 2023

Hi there, any idea when this would be released to PyPI?

aws-sdk-python-automation added a commit that referenced this pull request Oct 12, 2023
* release-1.31.63:
  Bumping version to 1.31.63
  Update to latest partitions and endpoints
  Update to latest models
  Remove deprecated `strict` poolmanager kwarg (#3036)
  add back line
  Sanitize __type metadata out of response parsing for tagged unions
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