[issue29742] asyncio get_extra_info() throws exception

Yury Selivanov added the comment: New changeset 2b27e2e6a35c3d3e369612b984017fe0d1bfcf0c by Yury Selivanov (Nikolay Kim) in branch 'master': bpo-29742: asyncio get_extra_info() throws exception (#525) https://github.com/python/cpython/commit/2b27e2e6a35c3d3e369612b984017fe0d1bfcf0c

[issue29742] asyncio get_extra_info() throws exception

Yury Selivanov added the comment: New changeset 99f8d33a94c8951fd6b2d5f079d20be698150993 by Yury Selivanov in branch '3.6': bpo-29742: asyncio get_extra_info() throws exception (#525) (#645) https://github.com/python/cpython/commit/99f8d33a94c8951fd6b2d5f079d20be698150993 --

[issue29742] asyncio get_extra_info() throws exception

Yury Selivanov added the comment: New changeset db522dc86294a99f46a63cd5b54c7c4ab8017c96 by Yury Selivanov in branch '3.5': bpo-29742: asyncio get_extra_info() throws exception (#525) (#646) https://github.com/python/cpython/commit/db522dc86294a99f46a63cd5b54c7c4ab8017c96 --

[issue29742] asyncio get_extra_info() throws exception

Changes by Larry Hastings : -- pull_requests: +596 ___ Python tracker ___ ___

[issue29742] asyncio get_extra_info() throws exception

Changes by Yury Selivanov : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___

[issue29742] asyncio get_extra_info() throws exception

Changes by Yury Selivanov : -- pull_requests: +534 ___ Python tracker ___ ___

[issue29742] asyncio get_extra_info() throws exception

Changes by Yury Selivanov : -- pull_requests: +535 ___ Python tracker ___ ___

[issue29742] asyncio get_extra_info() throws exception

Changes by Guido van Rossum : -- nosy: -gvanrossum ___ Python tracker ___ ___

[issue29742] asyncio get_extra_info() throws exception

Changes by Nikolay Kim : -- pull_requests: +438 ___ Python tracker ___ ___

[issue29742] asyncio get_extra_info() throws exception

Yury Selivanov added the comment: > Thanks for the report and PR, but your fix is not obviously correct. Not for this particular asyncio method though, see asyncio PEP [1]. But there's an actual bug in the PR, instead of returning `None` we should return `default`. [1]

[issue29742] asyncio get_extra_info() throws exception

Nikolay Kim added the comment: get_extra_info() returns optional transport information, I think it is ok to return None for closed transport. https://github.com/python/cpython/blob/master/Lib/asyncio/transports.py#L18 I propose this feature initially, during early tulip development but now I

[issue29742] asyncio get_extra_info() throws exception

R. David Murray added the comment: Thanks for the report and PR, but your fix is not obviously correct. In general exceptions are the way in Python that errors are reported, and asking for extra_info on a closed stream is an error. The exception raised is not clear, so perhaps the asyncio

[issue29742] asyncio get_extra_info() throws exception

Nikolay Kim added the comment: exception on get_extra_info() on closed ssl transport ``` Feb 18 13:18:09 btc electrumx_server.py[1732]: ERROR:ElectrumX:[15328] Traceback (most recent call last): Feb 18 13:18:09 btc electrumx_server.py[1732]: File

[issue29742] asyncio get_extra_info() throws exception

New submission from Nikolay Kim: https://github.com/python/asyncio/issues/494 -- messages: 289138 nosy: fafhrd91 priority: normal pull_requests: 435 severity: normal status: open title: asyncio get_extra_info() throws exception versions: Python 3.5, Python 3.6, Python 3.7