[issue32224] socket.create_connection needs to support full IPv6 argument

[Antoine Pitrou]

Antoine Pitrou  added the comment:

I think we still want the full tuple passed to connect().  We may want
to truncate the tuple when used for other purposes.

Le 07/12/2017 à 00:41, Matthew Stoltenberg a écrit :
> 
> Matthew Stoltenberg  added the comment:
> 
> The naive change is to have the host, port tuple assignment use only the 
> first two values from address. I can open a PR with that change (and update 
> the docstring for the function) if that's the right change. For reference, 
> the python socketpair implementation (if _socket is missing it) ignores 
> flow_info and scope_id, but I'm not sure that's the right thing to do.
> 
> --
> 
> ___
> Python tracker 
> 
> ___
>

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue32224] socket.create_connection needs to support full IPv6 argument

[Matthew Stoltenberg]

Matthew Stoltenberg  added the comment:

The naive change is to have the host, port tuple assignment use only the first 
two values from address. I can open a PR with that change (and update the 
docstring for the function) if that's the right change. For reference, the 
python socketpair implementation (if _socket is missing it) ignores flow_info 
and scope_id, but I'm not sure that's the right thing to do.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue32224] socket.create_connection needs to support full IPv6 argument

[Antoine Pitrou]

Antoine Pitrou  added the comment:

Also note that a regular socket.connect() call supports the full form address:

>>> sock = socket.socket(socket.AF_INET6, socket.SOCK_STREAM)
>>> sock.connect(('::1', 12345, 0, 0))

Matthew, are you interested in submitting a Github PR for this issue?

--
nosy: +gregory.p.smith, pitrou
stage:  -> needs patch
versions: +Python 2.7, Python 3.7 -Python 3.4

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue32224] socket.create_connection needs to support full IPv6 argument

[Matthew Stoltenberg]

Change by Matthew Stoltenberg :


--
title: socket.creaet_connection needs to support full IPv6 argument -> 
socket.create_connection needs to support full IPv6 argument

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com