Skip to content

Commit

Permalink
Remove obsolete comment (#873)
Browse files Browse the repository at this point in the history
(cherry picked from commit 6ba30da)
  • Loading branch information
sblondon authored and rthalley committed Jan 2, 2023
1 parent 1f46c2a commit 8452623
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions dns/query.py
Original file line number Diff line number Diff line change
Expand Up @@ -1032,13 +1032,7 @@ def tls(
where, port, source, source_port
)
if ssl_context is None and not sock:
# LGTM complains about this because the default might permit TLS < 1.2
# for compatibility, but the python documentation says that explicit
# versioning is deprecated, and that as of python 3.6 it will negotiate
# the highest version possible. We also set a minimum version when we
# can, even though this might require a future dnspython release if that
# version becomes deprecated.
ssl_context = ssl.create_default_context() # lgtm[py/insecure-protocol]
ssl_context = ssl.create_default_context()
ssl_context.minimum_version = ssl.TLSVersion.TLSv1_2
if server_hostname is None:
ssl_context.check_hostname = False
Expand Down

0 comments on commit 8452623

Please sign in to comment.