[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Change by Andrew Svetlov : -- resolution: -> fixed stage: patch review -> resolved status: open -> closed ___ Python tracker ___

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Andrew Svetlov added the comment: New changeset fd512d76456b65c529a5bc58d8cfe73e4a10de7a by Andrew Svetlov (Vincent Michel) in branch 'master': bpo-35065: Remove `StreamReaderProtocol._untrack_reader` (#10212) https://github.com/python/cpython/commit/fd512d76456b65c529a5bc58d8cfe73e4a10de7a

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Change by Vincent Michel : -- pull_requests: +9528 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list mailing list

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Vincent Michel added the comment: I found the culprit: https://github.com/python/cpython/blob/a05bef4f5be1bcd0df63ec0eb88b64fdde593a86/Lib/asyncio/streams.py#L350 The call to `_untrack_reader` is performed too soon. Closing the transport causes `protocol.connection_lost()` to be "called

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Vincent Michel added the comment: Hi Andrew! I reverted the commit associated with the following PR, and the hanging issue disappeared: https://github.com/python/cpython/pull/9201 I'll look into it. -- type: -> behavior ___ Python tracker

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

Andrew Svetlov added the comment: Hi Vincent! No, the hang is not intended behavior. Thanks for the report. I'll assign the issue to myself to don't miss the bug; but if you have a patch with a fix -- please don't hesitate to make a Pull Request for it. --

[issue35065] Reading received data from a closed TCP stream using `StreamReader.read` might hang forever

New submission from Vincent Michel : I'm not sure whether it is intended or not, but I noticed a change in the behavior of `StreamReader` between version 3.7 and 3.8. Basically, reading some received data from a closed TCP stream using `StreamReader.read` might hang forever, under certain