[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Michael Felt added the comment: I could not "fathom" the buildbot test results - however, a manual build of PR29170 on 3.9 works: I'll try 3.8, but then from master (assuming it is already part of master) -- and that works as well. Thanks for the quick update! --

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Nathaniel Smith added the comment: I think I fixed the buildbot issues in GH-20170, but I can't seem to reach the buildbot site right now, so it's hard to know for sure! -- versions: -Python 3.6 ___ Python tracker

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Nathaniel Smith added the comment: New changeset 58205a0217e91232cc1e945dbfe4e387d636eb76 by Nathaniel J. Smith in branch 'master': bpo-39148: fixup to account for IPV6_ENABLED being moved (GH-20170) https://github.com/python/cpython/commit/58205a0217e91232cc1e945dbfe4e387d636eb76

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Nathaniel Smith : -- pull_requests: +19469 pull_request: https://github.com/python/cpython/pull/20170 ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

miss-islington added the comment: New changeset 94d9c5e5afdee4f46be9d9faaa39d6be40c34849 by Miss Islington (bot) in branch '3.8': bpo-39148: enable ipv6 for datagrams in Proactor (GH-19121) https://github.com/python/cpython/commit/94d9c5e5afdee4f46be9d9faaa39d6be40c34849 --

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Michael Felt added the comment: Bot failed for AIX https://buildbot.python.org/all/#builders/227/builds/978 with: 0:07:11 Re-running test_asyncio in verbose mode Failed to import test module: test.test_asyncio.test_events Traceback (most recent call last): File

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by miss-islington : -- pull_requests: +19468 pull_request: https://github.com/python/cpython/pull/20169 ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

miss-islington added the comment: New changeset 442634c42fcaf31c636f693951a97734042c3e7b by Kjell Braden in branch 'master': bpo-39148: enable ipv6 for datagrams in Proactor (GH-19121) https://github.com/python/cpython/commit/442634c42fcaf31c636f693951a97734042c3e7b -- nosy:

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Alex Grönholm added the comment: The PR is still awaiting for a core developer to be reviewed. It's too bad we missed the 3.8.3 window, but perhaps this can get included in 3.9.0 at least. -- ___ Python tracker

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Alex Grönholm added the comment: Thanks, looks good to me now! -- ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: Fair enough, PR updated. -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by STINNER Victor : Removed file: https://bugs.python.org/file49089/readme.md20629842+super...@users.noreply.github.com ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by STINNER Victor : -- Removed message: https://bugs.python.org/msg367131 ___ Python tracker ___ ___ Python-bugs-list

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Marcin Wiśniewsk added the comment: >``># #7¡0.. ⁵ >⁵&^^} >7&£_86>> X.* >{[》》_`&&■●♤ $£/>}<¡#¤>⁵="⁵ -- nosy: +Marcin Wiśniewsk versions: +Python 3.6 Added file: https://bugs.python.org/file49089/readme.md20629842+super...@users.noreply.github.com

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Alex Grönholm added the comment: Oh, it's https://github.com/python/cpython/pull/19121. I think it would be prudent to add a test as well to make sure this doesn't happen again. -- ___ Python tracker

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Alex Grönholm added the comment: Which PR is it? -- ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: Please let me know if there is anything else I can do to get this going. -- ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: PR is up. Any chance we get this reviewed for inclusion in 3.9.0a6 / 3.8.3rc1? -- ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Kjell Braden : -- keywords: +patch pull_requests: +18482 stage: -> patch review pull_request: https://github.com/python/cpython/pull/19121 ___ Python tracker ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Steve Dower added the comment: Main thing we need is a PR (straightforward) and confirming that the relevant tests all run. We can use the buildbot fleet to make sure that all supported platforms have the necessary APIs, but it wouldn't surprise me if there's some weird edge cases here.

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Andrew Svetlov added the comment: Would somebody be a champion for the issue? -- ___ Python tracker ___ ___ Python-bugs-list

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: ./configure is not run on Windows, but I believe the define should go here: https://github.com/python/cpython/blob/8510f430781118d9b603c3a2f06945d6ebc5fe42/PC/pyconfig.h#L678 -- ___ Python tracker

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Alex Grönholm added the comment: Well, I found this: https://github.com/python/cpython/blob/3c97e1e457033bbb8bbe0b7198bd13fc794a12b0/configure.ac#L3278-L3279 Could it be that the official binaries were compiled without --enable-ipv6? -- ___ Python

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: Just to confirm, for some reason ENABLE_IPV6 is not set when compiling _overlapped. I'm not familiar enough with the windows build infrastructure to submit a PR though. -- versions: +Python 3.9 ___ Python tracker

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Kjell Braden added the comment: I tried some debugging with Python 3.9.0a4 and it looks like unparse_address in overlapped.c (https://github.com/python/cpython/blob/v3.9.0a4/Modules/overlapped.c#L689) raises the error. Almost seems like ENABLE_IPV6 is not set... --

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Kjell Braden : Removed file: https://bugs.python.org/file48996/udp_ipv6_server.py ___ Python tracker ___ ___ Python-bugs-list

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Kjell Braden : -- nosy: +kbr Added file: https://bugs.python.org/file48996/udp_ipv6_server.py ___ Python tracker ___ ___

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Nathaniel Smith : -- nosy: +njs ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

Change by Alex Grönholm : -- components: +asyncio nosy: +yselivanov ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue39148] DatagramProtocol + IPv6 does not work with ProactorEventLoop

New submission from Alex Grönholm : Receiving a UDP datagram using DatagramProtocol on the Proactor event loop results in error_received() being called with WinError 87 (Invalid Parameter). The low-level sock_recv() works fine, but naturally loses the sender address information. The attached