[issue23749] asyncio missing wrap_socket (starttls)

STINNER Victor added the comment: I found a race condition in START TLS: bpo-33674. I'm fixing it (I'm just waiting to merge my PR which has already been approved). -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

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

[issue23749] asyncio missing wrap_socket (starttls)

Andrew Svetlov added the comment: Is the issue done? -- ___ Python tracker ___

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: I have a feeling that using threads+IO+asyncio makes the test too unstable on some Windows buildbots. I'll rewrite start-tls tests without using threads. -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

STINNER Victor added the comment: test_start_tls_server_1() still fails randomly. Example on AppVeyor on my PR 5423: https://ci.appveyor.com/project/python/cpython/build/3.7build11472 ERROR: test_start_tls_server_1

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: New changeset 1e5b25b8c0c45ccfd58da2cb82fdf231c6823fef by Yury Selivanov in branch 'master': bpo-23749: Make start-tls tests more stable on win7 buildbot (GH-5409)

[issue23749] asyncio missing wrap_socket (starttls)

Change by Yury Selivanov : -- pull_requests: +5242 stage: resolved -> patch review ___ Python tracker ___

[issue23749] asyncio missing wrap_socket (starttls)

STINNER Victor added the comment: > Tests fail on x86 Windows7 3.x: I created a more specific issue: bpo-32645, test_asyncio: TLS tests fail on "x86 Windows7" buildbot. -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

STINNER Victor added the comment: SelectorStartTLSTests failed once on x86 Tiger 3.x in build 453, but then passed, no idea why. http://buildbot.python.org/all/#/builders/30/builds/453 == ERROR:

[issue23749] asyncio missing wrap_socket (starttls)

STINNER Victor added the comment: http://buildbot.python.org/all/#/builders/58/builds/435 Tests fail on x86 Windows7 3.x: == ERROR: test_start_tls_client_1

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: > I think the feature is significant enough for a What's New entry. Sure, Elvis and I will go through all NEWS items when it's time for what's new ;) -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

Barry A. Warsaw added the comment: @yselivanov - thanks for adding this, it's a huge win. I think the feature is significant enough for a What's New entry. -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

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

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: New changeset f111b3dcb414093a4efb9d74b69925e535ddc470 by Yury Selivanov in branch 'master': bpo-23749: Implement loop.start_tls() (#5039) https://github.com/python/cpython/commit/f111b3dcb414093a4efb9d74b69925e535ddc470 --

[issue23749] asyncio missing wrap_socket (starttls)

Change by Yury Selivanov : -- pull_requests: +4923 stage: needs patch -> patch review ___ Python tracker ___

[issue23749] asyncio missing wrap_socket (starttls)

Christian Heimes added the comment: I'm removing myself and drop the SSL component. It's really a feature request for asyncio. -- assignee: christian.heimes -> yselivanov components: -SSL ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

Barry A. Warsaw added the comment: I'm very interested in this because, even though we do support STARTTLS in aiosmtpd, it's a hack using non-public symbols, and we have a hidden traceback! (I.e. one that doesn't cause the test suite to fail, but only shows up when clients disconnect.)

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Barry A. Warsaw : -- nosy: +barry ___ Python tracker ___ ___ Python-bugs-list

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Guido van Rossum : -- nosy: -gvanrossum ___ Python tracker ___ ___

[issue23749] asyncio missing wrap_socket (starttls)

Changes by François : -- nosy: +Frzk ___ Python tracker ___ ___ Python-bugs-list

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: With the latest change it's possible to implement starttls as a separate package on PyPI, or even by copying/pasting a small snipped of code in your project. It's expected that we'll figure out the API design for starttls during 3.6, so that we can add it in

[issue23749] asyncio missing wrap_socket (starttls)

Roundup Robot added the comment: New changeset 3771a6326725 by Yury Selivanov in branch '3.5': asyncio: Add "call_connection_made" arg to SSLProtocol.__init__ https://hg.python.org/cpython/rev/3771a6326725 New changeset 3e6739e5c2d0 by Yury Selivanov in branch '3.6': Merge 3.5 (issue #23749)

[issue23749] asyncio missing wrap_socket (starttls)

Alex Grönholm added the comment: So is this going to make it into 3.6...? -- nosy: +Alex Grönholm ___ Python tracker ___

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Christian Heimes : -- assignee: -> christian.heimes components: +SSL nosy: +christian.heimes versions: +Python 3.6, Python 3.7 -Python 3.4 ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: > yuri, did you saw guido added review on your patch? Yes. There are few more issues with the patch that I want to resolve before re-submitting it for another review. Will do it soon. -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

박세원 added the comment: https://bugs.python.org/review/23749/#msg1 yuri, did you saw guido added review on your patch? -- nosy: +박세원 ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: I'll create a PR on the GitHub for this. I like the proposed design, and I've implemented an SSL test micro-framework that we can use to test starttls in asyncio. -- ___ Python tracker

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Robert Siemer : -- nosy: +siemer ___ Python tracker ___ ___

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: Guido, Victor, any thoughts about the (proto-)patch? -- ___ Python tracker ___

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: Guido, Victor, Please find attached a first draft of the patch. It's a very early attempt (i.e. I'm not including unit tests/docstrings), and its primary purpose is to gather initial feedback. Some points: 1. As discussed earlier, the primary API point is

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: Here's an example client implementation with writer.start_tls() (taken from my debug code): @asyncio.coroutine def client(addr): reader, writer = yield from asyncio.open_connection( *addr, loop=loop)

[issue23749] asyncio missing wrap_socket (starttls)

Elizabeth Myers added the comment: > Therefore, the most convenient place to add the new API are *transports*. I had an inkling this was the case, but I didn't know how to go about the creation of a new protocol and transport pair. > I'm hesitant to add this API to Transport; it somehow feels

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Mathieu Sornay : -- nosy: +msornay ___ Python tracker ___ ___ Python-bugs-list

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Antoine Pitrou pit...@free.fr: -- nosy: -pitrou ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23749 ___ ___ Python-bugs-list mailing

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Alex Grönholm alex.gronholm+pyt...@nextday.fi: -- nosy: +alex.gronholm ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23749 ___ ___

[issue23749] asyncio missing wrap_socket (starttls)

Yury Selivanov added the comment: I'm working on porting pypostgresql (pure python postgresql driver) library to use asyncio as its underlying IO machinery. And it appears that PQ3 protocol starts as clear text, and then upgrades to use TLS (if server or connection configured so). I've been

[issue23749] asyncio missing wrap_socket (starttls)

Changes by Elizabeth Myers elizab...@interlinked.me: -- nosy: -Elizacat ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23749 ___ ___

[issue23749] asyncio missing wrap_socket (starttls)

Changes by STINNER Victor victor.stin...@gmail.com: -- title: asyncio missing wrap_socket - asyncio missing wrap_socket (starttls) ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue23749 ___