diff options
author | Yury Selivanov <yury@magic.io> | 2018-06-05 12:59:58 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-06-05 12:59:58 (GMT) |
commit | 415bc46a78e785f357c8960ae70f18a6b6cccbb6 (patch) | |
tree | 9b99fb2ad8eb0d343d1e3c8e061881ed5e657b9d /Lib/asyncio/base_events.py | |
parent | e9e397605789b2a67b67558fbbe756b7b88934f5 (diff) | |
download | cpython-415bc46a78e785f357c8960ae70f18a6b6cccbb6.zip cpython-415bc46a78e785f357c8960ae70f18a6b6cccbb6.tar.gz cpython-415bc46a78e785f357c8960ae70f18a6b6cccbb6.tar.bz2 |
bpo-33769: start_tls: Fix error message; cancel callbacks on error (GH-7403)
In addition to that, mark SSLTransport as "closed" in its "abort()" method to prevent bogus warnings.
Diffstat (limited to 'Lib/asyncio/base_events.py')
-rw-r--r-- | Lib/asyncio/base_events.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/Lib/asyncio/base_events.py b/Lib/asyncio/base_events.py index 34cc625..68a1ebe 100644 --- a/Lib/asyncio/base_events.py +++ b/Lib/asyncio/base_events.py @@ -1097,7 +1097,7 @@ class BaseEventLoop(events.AbstractEventLoop): if not getattr(transport, '_start_tls_compatible', False): raise TypeError( - f'transport {self!r} is not supported by start_tls()') + f'transport {transport!r} is not supported by start_tls()') waiter = self.create_future() ssl_protocol = sslproto.SSLProtocol( @@ -1111,13 +1111,15 @@ class BaseEventLoop(events.AbstractEventLoop): transport.pause_reading() transport.set_protocol(ssl_protocol) - self.call_soon(ssl_protocol.connection_made, transport) - self.call_soon(transport.resume_reading) + conmade_cb = self.call_soon(ssl_protocol.connection_made, transport) + resume_cb = self.call_soon(transport.resume_reading) try: await waiter except Exception: transport.close() + conmade_cb.cancel() + resume_cb.cancel() raise return ssl_protocol._app_transport |