➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2017-03-06 22:35 by fafhrd91, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 494 | fafhrd91, 2017-03-06 22:35 | ||
| PR 525 | merged | fafhrd91, 2017-03-06 23:36 | |
| PR 645 | merged | yselivanov, 2017-03-12 19:29 | |
| PR 646 | merged | yselivanov, 2017-03-12 19:30 | |
| PR 703 | larry, 2017-03-17 21:00 | ||
| Messages (8) | |||
|---|---|---|---|
| msg289138 - (view) | Author: Nikolay Kim (fafhrd91) * | Date: 2017-03-06 22:35 | |
https://github.com/python/asyncio/issues/494 |
|||
| msg289139 - (view) | Author: Nikolay Kim (fafhrd91) * | Date: 2017-03-06 22:37 | |
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 "/usr/local/lib/python3.5/dist-packages/electrumx-0.11.1-py3.5.egg/lib/jsonrpc.py", line 528, in process_single_request Feb 18 13:18:09 btc electrumx_server.py[1732]: result = await self.handle_payload(payload, self.request_handler) Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/local/lib/python3.5/dist-packages/electrumx-0.11.1-py3.5.egg/lib/jsonrpc.py", line 608, in handle_payload Feb 18 13:18:09 btc electrumx_server.py[1732]: return await handler(**kw_args) Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/local/lib/python3.5/dist-packages/electrumx-0.11.1-py3.5.egg/server/session.py", line 282, in banner Feb 18 13:18:09 btc electrumx_server.py[1732]: if self.is_tor(): Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/local/lib/python3.5/dist-packages/electrumx-0.11.1-py3.5.egg/server/session.py", line 259, in is_tor Feb 18 13:18:09 btc electrumx_server.py[1732]: peer_info = self.peer_info() Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/local/lib/python3.5/dist-packages/electrumx-0.11.1-py3.5.egg/lib/jsonrpc.py", line 764, in peer_info Feb 18 13:18:09 btc electrumx_server.py[1732]: return self.transport.get_extra_info('peername') Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/lib/python3.5/asyncio/sslproto.py", line 306, in get_extra_info Feb 18 13:18:09 btc electrumx_server.py[1732]: return self._ssl_protocol._get_extra_info(name, default) Feb 18 13:18:09 btc electrumx_server.py[1732]: File "/usr/lib/python3.5/asyncio/sslproto.py", line 537, in _get_extra_info Feb 18 13:18:09 btc electrumx_server.py[1732]: return self._transport.get_extra_info(name, default) Feb 18 13:18:09 btc electrumx_server.py[1732]: AttributeError: 'NoneType' object has no attribute 'get_extra_info' ``` |
|||
| msg289140 - (view) | Author: R. David Murray (r.david.murray) *  |
Date: 2017-03-06 22:44 | |
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 folks will want to improve it. Or perhaps I'm wrong and they will want to return None, but I'll be a bit surprised if that is the case :) |
|||
| msg289141 - (view) | Author: Nikolay Kim (fafhrd91) * | Date: 2017-03-06 22:49 | |
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 think it is not good api. |
|||
| msg289142 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2017-03-06 22:50 | |
> 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] https://www.python.org/dev/peps/pep-3156/#methods-for-all-transports |
|||
| msg290192 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2017-03-24 22:21 | |
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 |
|||
| msg290193 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2017-03-24 22:21 | |
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 |
|||
| msg290196 - (view) | Author: Yury Selivanov (yselivanov) * |
Date: 2017-03-24 22:21 | |
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 |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:43 | admin | set | github: 73928 |
| 2017-03-24 22:21:52 | yselivanov | set | messages: + msg290196 |
| 2017-03-24 22:21:26 | yselivanov | set | messages: + msg290193 |
| 2017-03-24 22:21:21 | yselivanov | set | messages: + msg290192 |
| 2017-03-17 21:00:33 | larry | set | pull_requests: + pull_request596 |
| 2017-03-12 21:06:52 | yselivanov | set | status: open -> closed resolution: fixed stage: resolved |
| 2017-03-12 19:30:20 | yselivanov | set | pull_requests: + pull_request535 |
| 2017-03-12 19:29:56 | yselivanov | set | pull_requests: + pull_request534 |
| 2017-03-06 23:40:52 | gvanrossum | set | nosy:
- gvanrossum |
| 2017-03-06 23:36:42 | fafhrd91 | set | pull_requests: + pull_request438 |
| 2017-03-06 22:50:39 | yselivanov | set | messages: + msg289142 |
| 2017-03-06 22:49:12 | fafhrd91 | set | messages: + msg289141 |
| 2017-03-06 22:44:24 | r.david.murray | set | nosy:
+ r.david.murray, gvanrossum, yselivanov messages: + msg289140 components: + asyncio |
| 2017-03-06 22:37:31 | fafhrd91 | set | messages: + msg289139 |
| 2017-03-06 22:35:43 | fafhrd91 | create | |