Skip to content

gh-100985: Consistently wrap IPv6 IP address during CONNECT #100986

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

Merged
merged 3 commits into from
Feb 17, 2024

Conversation

derekhiggins
Copy link
Contributor

@derekhiggins derekhiggins commented Jan 12, 2023

Update _get_hostport to always remove square brackets from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".

@ghost
Copy link

ghost commented Jan 12, 2023

All commit authors signed the Contributor License Agreement.
CLA signed

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
@arhadthedev arhadthedev added the stdlib Python modules in the Lib dir label Feb 25, 2023
Copy link
Member

@serhiy-storchaka serhiy-storchaka left a comment

Choose a reason for hiding this comment

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

LGTM, with some nitpicks.

@serhiy-storchaka serhiy-storchaka enabled auto-merge (squash) February 17, 2024 09:51
@serhiy-storchaka serhiy-storchaka added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Feb 17, 2024
@serhiy-storchaka serhiy-storchaka merged commit 465db27 into python:main Feb 17, 2024
@miss-islington-app
Copy link

Thanks @derekhiggins for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 17, 2024
…thonGH-100986)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
(cherry picked from commit 465db27)

Co-authored-by: Derek Higgins <derekh@redhat.com>
@miss-islington-app
Copy link

Sorry, @derekhiggins and @serhiy-storchaka, I could not cleanly backport this to 3.11 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 465db27cb983084e718a1fd9519b2726c96935cb 3.11

@bedevere-app
Copy link

bedevere-app bot commented Feb 17, 2024

GH-115591 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Feb 17, 2024
serhiy-storchaka pushed a commit that referenced this pull request Feb 17, 2024
…H-100986) (GH-115591)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
(cherry picked from commit 465db27)

Co-authored-by: Derek Higgins <derekh@redhat.com>
serhiy-storchaka pushed a commit to serhiy-storchaka/cpython that referenced this pull request Feb 17, 2024
…ECT (pythonGH-100986)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
(cherry picked from commit 465db27)

Co-authored-by: Derek Higgins <derekh@redhat.com>
@bedevere-app
Copy link

bedevere-app bot commented Feb 17, 2024

GH-115606 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label Feb 17, 2024
@serhiy-storchaka serhiy-storchaka removed their assignment Feb 17, 2024
serhiy-storchaka added a commit that referenced this pull request Feb 17, 2024
…H-100986) (GH-115606)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
(cherry picked from commit 465db27)

Co-authored-by: Derek Higgins <derekh@redhat.com>
woodruffw pushed a commit to woodruffw-forks/cpython that referenced this pull request Mar 4, 2024
…thonGH-100986)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
diegorusso pushed a commit to diegorusso/cpython that referenced this pull request Apr 17, 2024
…thonGH-100986)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
LukasWoodtli pushed a commit to LukasWoodtli/cpython that referenced this pull request Jan 22, 2025
…thonGH-100986)

Update _get_hostport to always remove square brackets
from IPv6 addresses. Then add them if needed
in "CONNECT .." and "Host: ".
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stdlib Python modules in the Lib dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy