Skip to content

Prevent HTTPResponse.getheader() is deprecated warning#206

Merged
keelerm84 merged 1 commit into
launchdarkly:mainfrom
mnito:fix_getheader_deprecation_warning
May 3, 2023
Merged

Prevent HTTPResponse.getheader() is deprecated warning#206
keelerm84 merged 1 commit into
launchdarkly:mainfrom
mnito:fix_getheader_deprecation_warning

Conversation

@mnito

@mnito mnito commented May 2, 2023

Copy link
Copy Markdown
Contributor

Requirements

  • I have added test coverage for new or changed functionality (already covered adequately by tests)
  • I have followed the repository's pull request submission guidelines
  • I have validated my changes against all supported platform versions

Related issues

Provide links to any issues in this repository or elsewhere relating to this pull request.

Describe the solution you've provided

Prevents "DeprecationWarning: HTTPResponse.getheader() is deprecated and will be removed in urllib3 v2.1.0. Instead use HTTPResponse.headers.get(name, default)" from happening by updating relevant code to the supported way (HTTPResponse.headers.get)

Describe alternatives you've considered

Provide a clear and concise description of any alternative solutions or features you've considered.

Additional context

Refer to urllib3/urllib3#2814

@keelerm84 keelerm84 merged commit da12ac1 into launchdarkly:main May 3, 2023
@keelerm84

Copy link
Copy Markdown
Member

Thank you for your contribution! I will work on getting a release out as soon as I can with this fix.

LaunchDarklyReleaseBot pushed a commit that referenced this pull request May 3, 2023
@keelerm84

Copy link
Copy Markdown
Member

Fixed in 8.1.3

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.

2 participants