[issue34971] add support for tls/ssl sessions in asyncio
Rémi Cardona added the comment: Anything I can do to get the ball rolling? Let me know who to get in touch with and *how*, and I will. Thanks -- ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue29930] Waiting for asyncio.StreamWriter.drain() twice in parallel raises an AssertionError when the transport stopped writing
Rémi Cardona added the comment: Got hit by this too, many times as I'm working with 3G devices (slow write speeds). As for "drain()", I'd say it should work like a fence/barrier: to let you know that the bytes in the buffer up to when the call was issued have been successfully written on the socket. I'll see what I can cook up. Cheers -- nosy: +RemiCardona ___ Python tracker <http://bugs.python.org/issue29930> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue34971] add support for tls/ssl sessions in asyncio
New submission from Rémi Cardona : cpython has had TLS session support since 3.6, using the SSLContext.wrap_* methods. Unfortunately, this support is not available when using asyncio's create_connection. While I've managed to monkeypatch asyncio.sslproto._SSLPipe from my own code (it's a filthy hack but it's short and it gets the job done) running on 3.6.6, I feel this should be properly supported out of the box. A patch is ready (tests work), a github PR will be created shortly. Notes in no particular order: - argument and attribute naming is all over the place, but I could not decide between "sslsession" (matching "sslcontext") and "ssl_session" (matching "ssl_handshake_timeout") so I just picked one - tested on jessie (with openssl 1.0.2 from jessie-backports) and on gentoo - the new asyncio tests added in the patch are adapted from test_ssl.py's test_session, with the server-side stats left out. I felt they were not useful if one assumes that the hard work is done by SSLContext.wrap_*. - I did not reuse test_asyncio.utils.run_test_server which AIUI creates a new server-side context for each incoming connection, thus breaking sessions completely TIA for considering this bug and patch -- components: asyncio messages: 327638 nosy: RemiCardona, asvetlov, yselivanov priority: normal severity: normal status: open title: add support for tls/ssl sessions in asyncio type: enhancement ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue34971] add support for tls/ssl sessions in asyncio
Change by Rémi Cardona : -- keywords: +patch pull_requests: +9214 stage: -> patch review ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue34971] add support for tls/ssl sessions in asyncio
Rémi Cardona added the comment: Hi Andrew, How should I proceed? What's the best avenue to get in touch with Yuri? Thanks -- ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue34971] add support for tls/ssl sessions in asyncio
Rémi Cardona added the comment: Hi Christian, Could you tell me more about this new openssl API? Right now my patch works with whatever the ssl module provides. Are you suggesting the ssl module is in some way incomplete? Would supporting TLS 1.3 sessions be incompatible with the current session API? I'd like to help wherever possible, but I'm probably missing some context and/or knowledge around all things TLS in cpython. Thanks -- ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue34971] add support for tls/ssl sessions in asyncio
Rémi Cardona added the comment: So, IOW, the ssl module needs a good shakeup wrt TLS 1.3 sessions before any asyncio work can be merged. Am I getting this right? In which case, a whole other issue/PR is needed and possibly better folks than me. I try to stay clear of low-level crypto APIs because I don't trust myself to get things right. Well… I certainly can look at it, but I fear I may be punching above my weight with this. -- ___ Python tracker <https://bugs.python.org/issue34971> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue29309] Interpreter hang when interrupting a loop.run_in_executor() future
Changes by Rémi Cardona : -- nosy: +RemiCardona ___ Python tracker <http://bugs.python.org/issue29309> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue26081] Implement asyncio Future in C to improve performance
Changes by Rémi Cardona : -- nosy: +RemiCardona ___ Python tracker <http://bugs.python.org/issue26081> ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
