Skip to content

bpo-33654: Support BufferedProtocol in set_protocol() and start_tls() #7130

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 and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
May 28, 2018

Conversation

1st1
Copy link
Member

@1st1 1st1 commented May 26, 2018

Copy link
Contributor

@asvetlov asvetlov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@asvetlov
Copy link
Contributor

hmm, windows tests are failing

@1st1
Copy link
Member Author

1st1 commented May 26, 2018

Yes, i'll be looking into win issues before merging.

@1st1
Copy link
Member Author

1st1 commented May 27, 2018

@asvetlov please take a look at the 0238da4 -- I think I fixed the windows issue

@asvetlov
Copy link
Contributor

Wow, nice.
Waiting for merge.

@1st1 1st1 merged commit dbf1022 into python:master May 28, 2018
@miss-islington
Copy link
Contributor

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

@1st1 1st1 deleted the fixbuf branch May 28, 2018 18:31
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 28, 2018
…pythonGH-7130)

In this commit:

* Support BufferedProtocol in set_protocol() and start_tls()
* Fix proactor to cancel readers reliably
* Update tests to be compatible with OpenSSL 1.1.1
* Clarify BufferedProtocol docs
* Bump TLS tests timeouts to 60 seconds; eliminate possible race from start_serving
* Rewrite test_start_tls_server_1
(cherry picked from commit dbf1022)

Co-authored-by: Yury Selivanov <yury@magic.io>
@bedevere-bot
Copy link

GH-7164 is a backport of this pull request to the 3.7 branch.

miss-islington added a commit that referenced this pull request May 28, 2018
…GH-7130)

In this commit:

* Support BufferedProtocol in set_protocol() and start_tls()
* Fix proactor to cancel readers reliably
* Update tests to be compatible with OpenSSL 1.1.1
* Clarify BufferedProtocol docs
* Bump TLS tests timeouts to 60 seconds; eliminate possible race from start_serving
* Rewrite test_start_tls_server_1
(cherry picked from commit dbf1022)

Co-authored-by: Yury Selivanov <yury@magic.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants