[issue34975] start_tls() difficult when using asyncio.start_server()

Gregory P. Smith added the comment: https://discuss.python.org/t/need-reconsideration-of-bpo-34975-add-start-tls-method-to-streams-api/14720 would like to see this reconsidered. reopening. -- nosy: +gregory.p.smith resolution: wont fix -> stage: resolved -> status: closed -> open

[issue34975] start_tls() difficult when using asyncio.start_server()

Andrew Svetlov added the comment: I think it should be closed; Yuri thinks that current streams API is frozen for the sake of shiny brand new streams (don't exist yet). -- resolution: -> wont fix status: open -> closed ___ Python tracker

[issue34975] start_tls() difficult when using asyncio.start_server()

Ian Good added the comment: #36889 was reverted, so this is not resolved. I'm guessing this needs to be moved to 3.9 now too. Is my original PR worth revisiting? https://github.com/python/cpython/pull/13143/files -- resolution: fixed -> status: closed -> open

[issue34975] start_tls() difficult when using asyncio.start_server()

Andrew Svetlov added the comment: Fixed by #36889 -- resolution: -> fixed stage: patch review -> resolved status: open -> closed superseder: -> Merge StreamWriter and StreamReader into just asyncio.Stream ___ Python tracker

[issue34975] start_tls() difficult when using asyncio.start_server()

Andrew Svetlov added the comment: I believe stream transport should be replaced. If you have time please do this change. Also please take a look at #36889 for merging StreamWriter and StreamReader. It can simplify the replacement strategy. Anyway, please keep your PR open at least. The plan

[issue34975] start_tls() difficult when using asyncio.start_server()

Ian Good added the comment: I added start_tls() to StreamWriter. My implementation returns a new StreamWriter that should be used from then on, but it could be adapted to modify the current writer in-place (let me know). I've added docs, an integration test, and done some additional

[issue34975] start_tls() difficult when using asyncio.start_server()

Change by Ian Good : -- keywords: +patch pull_requests: +13056 stage: -> patch review ___ Python tracker ___ ___ Python-bugs-list

[issue34975] start_tls() difficult when using asyncio.start_server()

Yury Selivanov added the comment: One thing: I'm -1 on adding starttls to current stream api; let's add it only to the new one (same for sendfile) -- ___ Python tracker ___

[issue34975] start_tls() difficult when using asyncio.start_server()

Andrew Svetlov added the comment: Thanks for raising the problem. I'm in the middle of streams API refactoring. https://github.com/asvetlov/cpython/blob/async-streams/Lib/asyncio/streams.py#L801-L812 is a draft. A help is very appreciated. Would you pick up this snippet and make a pull

[issue34975] start_tls() difficult when using asyncio.start_server()

New submission from Ian Good : There does not seem to be a public API for replacing the transport of the StreamReader / StreamWriter provided to the callback of a call to asyncio.start_server(). The only way I have found to use the new SSL transport is to update protected members of the