Skip to content

[3.7] bpo-33618: Enable TLS 1.3 in tests (GH-7079)#7082

Merged
tiran merged 1 commit intopython:3.7from
miss-islington:backport-529525f-3.7
May 23, 2018
Merged

[3.7] bpo-33618: Enable TLS 1.3 in tests (GH-7079)#7082
tiran merged 1 commit intopython:3.7from
miss-islington:backport-529525f-3.7

Conversation

@miss-islington
Copy link
Contributor

@miss-islington miss-islington commented May 23, 2018

TLS 1.3 behaves slightly different than TLS 1.2. Session tickets and TLS
client cert auth are now handled after the initialy handshake. Tests now
either send/recv data to trigger session and client certs. Or tests
ignore ConnectionResetError / BrokenPipeError on the server side to
handle clients that force-close the socket fd.

To test TLS 1.3, OpenSSL 1.1.1-pre7-dev (git master + OpenSSL PR
openssl/openssl#6340) is required.

Signed-off-by: Christian Heimes [email protected]
(cherry picked from commit 529525f)

Co-authored-by: Christian Heimes [email protected]

https://bugs.python.org/issue33618

TLS 1.3 behaves slightly different than TLS 1.2. Session tickets and TLS
client cert auth are now handled after the initialy handshake. Tests now
either send/recv data to trigger session and client certs. Or tests
ignore ConnectionResetError / BrokenPipeError on the server side to
handle clients that force-close the socket fd.

To test TLS 1.3, OpenSSL 1.1.1-pre7-dev (git master + OpenSSL PR
openssl/openssl#6340) is required.

Signed-off-by: Christian Heimes <[email protected]>
(cherry picked from commit 529525f)

Co-authored-by: Christian Heimes <[email protected]>
@miss-islington
Copy link
Contributor Author

@tiran: Backport status check is done, and it's a success ✅ .

@tiran tiran merged commit 72ef4fc into python:3.7 May 23, 2018
@miss-islington
Copy link
Contributor Author

Thanks, @tiran!

@miss-islington miss-islington deleted the backport-529525f-3.7 branch May 23, 2018 20:49
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.

4 participants