[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Change by Nic Watson : -- nosy: +jnwatson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Change by Yury Selivanov : -- pull_requests: +6842 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Yury Selivanov added the comment: > asyncio is picking ipv6 because of this code: > https://github.com/python/cpython/blob/a445feb72902e4a3c5ae712f0c289309e1580d52/Lib/asyncio/base_events.py#L1334-L1340 > despite my host not actually having an ipv6 network hooked up. Which should be fine;

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Yury Selivanov added the comment: Does anybody wants to make a PR to fix this? -- ___ Python tracker ___ ___ Python-bugs-list

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Cecile Tonglet added the comment: I see that the patch hasn't been applied to master on GitHub. Is there anything else expected from me on this ticket? -- ___ Python tracker

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Anthony Sottile added the comment: Applying this patch makes the tests pass for me, but I don't think the patch is appropriate (just hides the bug): ``` $ git diff diff --git a/Lib/test/test_asyncio/test_unix_events.py b/Lib/test/test_asyncio/test_unix_events.py index

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Anthony Sottile added the comment: Actually, my issue seems to be something more strange. The host being passed in is `localhost` which resolves to: ``` >>> pprint.pprint(socket.getaddrinfo('localhost', 80)) [(, , 6, '', ('127.0.0.1', 80)), (, , 17, '',

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Anthony Sottile added the comment: Seeing this as well when running the cpython test suite in docker: ``` $ ./python -m test.test_asyncio ... [18 similar traces omitted] == ERROR:

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Antoine Pitrou added the comment: Cécile, thank you. The reason the message is still displayed is that I turned the error into a warning (in other words, it is expected). -- ___ Python tracker

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Cecile Tonglet added the comment: Sure! It seems to work, the process returns an exit code of 0 and I see no traceback but the message is still displayed in the terminal. (Also I did something weird because your patch applies on branch master and I ran it with Python 3.6... I suppose it

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Antoine Pitrou added the comment: Cécile, could you try the following patch? I have no easy way to test here. -- keywords: +patch Added file: http://bugs.python.org/file47026/eaddrnotavail_asyncio.patch ___ Python tracker

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Yury Selivanov added the comment: > Better than trying to detect IPv6 compatibility beforehand would probably to > recognize the error and simply ignore it. +1 -- ___ Python tracker

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

Antoine Pitrou added the comment: Better than trying to detect IPv6 compatibility beforehand would probably to recognize the error and simply ignore it. Note: errno 99 is EADDRNOTAVAIL. Something like this could work (untested): diff --git a/Lib/asyncio/base_events.py

[issue30945] loop.create_server does not detect if the interface is IPv6 enabled

New submission from Cecile Tonglet: The IPv6 detection in asyncio.base_events.create_server only detect if IPv6 is available instead of checking if the interface can actually support it. I noticed that by using Python in a Docker container (example code to reproduce in attachment): docker