Skip to content

Commit 5905f27

Browse files
fantix1st1
authored andcommitted
prefer PROTOCOL_TLS than PROTOCOL_SSLv23
1 parent d9ad177 commit 5905f27

File tree

4 files changed

+18
-5
lines changed

4 files changed

+18
-5
lines changed

examples/bench/echoclient.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,10 @@
2929
client_context = None
3030
if args.ssl:
3131
print('with SSL')
32-
client_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
32+
if hasattr(ssl, 'PROTOCOL_TLS'):
33+
client_context = ssl.SSLContext(ssl.PROTOCOL_TLS)
34+
else:
35+
client_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
3336
if hasattr(client_context, 'check_hostname'):
3437
client_context.check_hostname = False
3538
client_context.verify_mode = ssl.CERT_NONE

examples/bench/echoserver.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,10 @@ async def print_debug(loop):
143143
server_context = None
144144
if args.ssl:
145145
print('with SSL')
146-
server_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
146+
if hasattr(ssl, 'PROTOCOL_TLS'):
147+
server_context = ssl.SSLContext(ssl.PROTOCOL_TLS)
148+
else:
149+
server_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
147150
server_context.load_cert_chain(
148151
(pathlib.Path(__file__).parent.parent.parent /
149152
'tests' / 'certs' / 'ssl_cert.pem'),

tests/test_tcp.py

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1651,13 +1651,17 @@ async def client(addr):
16511651
self.fail("unexpected call to connection_made()")
16521652

16531653
def test_ssl_connect_accepted_socket(self):
1654-
server_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
1654+
if hasattr(ssl, 'PROTOCOL_TLS'):
1655+
proto = ssl.PROTOCOL_TLS
1656+
else:
1657+
proto = ssl.PROTOCOL_SSLv23
1658+
server_context = ssl.SSLContext(proto)
16551659
server_context.load_cert_chain(self.ONLYCERT, self.ONLYKEY)
16561660
if hasattr(server_context, 'check_hostname'):
16571661
server_context.check_hostname = False
16581662
server_context.verify_mode = ssl.CERT_NONE
16591663

1660-
client_context = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
1664+
client_context = ssl.SSLContext(proto)
16611665
if hasattr(server_context, 'check_hostname'):
16621666
client_context.check_hostname = False
16631667
client_context.verify_mode = ssl.CERT_NONE

uvloop/_testbase.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -271,7 +271,10 @@ def find_free_port(start_from=50000):
271271
class SSLTestCase:
272272

273273
def _create_server_ssl_context(self, certfile, keyfile=None):
274-
sslcontext = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
274+
if hasattr(ssl, 'PROTOCOL_TLS'):
275+
sslcontext = ssl.SSLContext(ssl.PROTOCOL_TLS)
276+
else:
277+
sslcontext = ssl.SSLContext(ssl.PROTOCOL_SSLv23)
275278
sslcontext.options |= ssl.OP_NO_SSLv2
276279
sslcontext.load_cert_chain(certfile, keyfile)
277280
return sslcontext

0 commit comments

Comments
 (0)