Skip to content

Commit

Permalink
Merge pull request #2725 from bdarnell/ssl-win-py374
Browse files Browse the repository at this point in the history
test: Disable TLS 1.3 in one test
  • Loading branch information
bdarnell authored Aug 11, 2019
2 parents 444f4cd + 4172483 commit 131eee0
Showing 1 changed file with 10 additions and 3 deletions.
13 changes: 10 additions & 3 deletions tornado/test/iostream_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -973,9 +973,16 @@ def connect_to_server(self, server_cls):
server = server_cls(ssl_options=_server_ssl_options())
server.add_socket(sock)

client = SSLIOStream(
socket.socket(), ssl_options=dict(cert_reqs=ssl.CERT_NONE)
)
ssl_ctx = ssl.create_default_context(ssl.Purpose.SERVER_AUTH)
ssl_ctx.check_hostname = False
ssl_ctx.verify_mode = ssl.CERT_NONE
# These tests fail with ConnectionAbortedErrors with TLS
# 1.3 on windows python 3.7.4 (which includes an upgrade
# to openssl 1.1.c. Other platforms might be affected with
# newer openssl too). Disable it until we figure out
# what's up.
ssl_ctx.options |= getattr(ssl, "OP_NO_TLSv1_3", 0)
client = SSLIOStream(socket.socket(), ssl_options=ssl_ctx)
yield client.connect(("127.0.0.1", port))
self.assertIsNotNone(client.socket.cipher())
finally:
Expand Down

0 comments on commit 131eee0

Please sign in to comment.