bpo-33674: asyncio: Fix SSLProtocol race by vstinner · Pull Request #7175 · python/cpython (original) (raw)

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Conversation4 Commits2 Checks0 Files changed

Conversation

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

vstinner

Fix a race condition in SSLProtocol.connection_made() of
asyncio.sslproto: start immediately the handshake instead of using
call_soon(). Previously, data_received() could be called before the
handshake started, causing the handshake to hang or fail.

https://bugs.python.org/issue33674

@vstinner

Fix a race condition in SSLProtocol.connection_made() of asyncio.sslproto: start immediately the handshake instead of using call_soon(). Previously, data_received() could be called before the handshake started, causing the handshake to hang or fail.

1st1

1st1 approved these changes May 28, 2018

@vstinner

Create self._handshake_timeout_handle before calling _process_write_backlog().

1st1 added a commit to MagicStack/uvloop that referenced this pull request

May 28, 2018

@1st1

1st1 added a commit to MagicStack/uvloop that referenced this pull request

May 28, 2018

@1st1

@miss-islington

Thanks @vstinner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.6, 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

May 28, 2018

@vstinner @miss-islington

Fix a race condition in SSLProtocol.connection_made() of asyncio.sslproto: start immediately the handshake instead of using call_soon(). Previously, data_received() could be called before the handshake started, causing the handshake to hang or fail. (cherry picked from commit be00a55)

Co-authored-by: Victor Stinner vstinner@redhat.com

@bedevere-bot

@miss-islington

Sorry, @vstinner, I could not cleanly backport this to 3.6 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker be00a5583a2cb696335c527b921d1868266a42c6 3.6

@bedevere-bot

ned-deily pushed a commit that referenced this pull request

May 29, 2018

Fix a race condition in SSLProtocol.connection_made() of asyncio.sslproto: start immediately the handshake instead of using call_soon(). Previously, data_received() could be called before the handshake started, causing the handshake to hang or fail. (cherry picked from commit be00a55)

Co-authored-by: Victor Stinner vstinner@redhat.com

1st1 pushed a commit that referenced this pull request

May 29, 2018

@vstinner @1st1

Fix a race condition in SSLProtocol.connection_made() of asyncio.sslproto: start immediately the handshake instead of using call_soon(). Previously, data_received() could be called before the handshake started, causing the handshake to hang or fail.

(cherry picked from commit be00a55)

Olaf1022 pushed a commit to Olaf1022/Ultra-fast-asyncio that referenced this pull request

Aug 13, 2023

@1st1