From 91c8678d67bd5b726f001f3e2a0f6567a881ad2f Mon Sep 17 00:00:00 2001 From: python273 Date: Sat, 27 Aug 2022 01:23:29 +0300 Subject: [PATCH 1/2] gh-90467: StreamReaderProtocol - add strong reference to created task --- Lib/asyncio/streams.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Lib/asyncio/streams.py b/Lib/asyncio/streams.py index 9a169035de8865..614b2cda60682f 100644 --- a/Lib/asyncio/streams.py +++ b/Lib/asyncio/streams.py @@ -205,6 +205,7 @@ def __init__(self, stream_reader, client_connected_cb=None, loop=None): self._strong_reader = stream_reader self._reject_connection = False self._stream_writer = None + self._task = None self._transport = None self._client_connected_cb = client_connected_cb self._over_ssl = False @@ -247,7 +248,7 @@ def connection_made(self, transport): res = self._client_connected_cb(reader, self._stream_writer) if coroutines.iscoroutine(res): - self._loop.create_task(res) + self._task = self._loop.create_task(res) self._strong_reader = None def connection_lost(self, exc): @@ -265,6 +266,7 @@ def connection_lost(self, exc): super().connection_lost(exc) self._stream_reader_wr = None self._stream_writer = None + self._task = None self._transport = None def data_received(self, data): From 83a816de87ed15668362ada10ccd059261029ea1 Mon Sep 17 00:00:00 2001 From: python273 Date: Sat, 27 Aug 2022 14:39:48 +0300 Subject: [PATCH 2/2] Add blurb and ACKS --- Misc/ACKS | 2 ++ .../next/Library/2022-08-27-14-38-49.gh-issue-90467.VOOB0p.rst | 2 ++ 2 files changed, 4 insertions(+) create mode 100644 Misc/NEWS.d/next/Library/2022-08-27-14-38-49.gh-issue-90467.VOOB0p.rst diff --git a/Misc/ACKS b/Misc/ACKS index daa01c1a0f1dbb..0edea9219d05ef 100644 --- a/Misc/ACKS +++ b/Misc/ACKS @@ -1442,6 +1442,7 @@ Ram Rachum Jeffrey Rackauckas Jérôme Radix Burton Radons +Kirill (python273) R. Abhilash Raj Shorya Raj Ajith Ramachandran @@ -1987,6 +1988,7 @@ Gordon Worley Darren Worrall Thomas Wouters Daniel Wozniak +Simon Wrede Marcin Niemira Wei Wu Heiko Wundram diff --git a/Misc/NEWS.d/next/Library/2022-08-27-14-38-49.gh-issue-90467.VOOB0p.rst b/Misc/NEWS.d/next/Library/2022-08-27-14-38-49.gh-issue-90467.VOOB0p.rst new file mode 100644 index 00000000000000..282c0e76a8c81c --- /dev/null +++ b/Misc/NEWS.d/next/Library/2022-08-27-14-38-49.gh-issue-90467.VOOB0p.rst @@ -0,0 +1,2 @@ +Fix :class:`asyncio.streams.StreamReaderProtocol` to keep a strong reference +to the created task, so that it's not garbage collected