Skip to content

tls: use auto-negotiated highest version#3007

Merged
milas merged 1 commit intodocker:masterfrom
milas:py-tls-deprecation
Jul 26, 2022
Merged

tls: use auto-negotiated highest version#3007
milas merged 1 commit intodocker:masterfrom
milas:py-tls-deprecation

Conversation

@milas
Copy link
Copy Markdown
Contributor

@milas milas commented Jul 26, 2022

Specific TLS versions are deprecated in latest Python, which
causes test failures due to treating deprecation errors as
warnings.

Luckily, the fix here is straightforward: we can eliminate some
custom version selection logic by using PROTOCOL_TLS_CLIENT,
which is the recommended method and will select the highest TLS
version supported by both client and server.

Specific TLS versions are deprecated in latest Python, which
causes test failures due to treating deprecation errors as
warnings.

Luckily, the fix here is straightforward: we can eliminate some
custom version selection logic by using `PROTOCOL_TLS_CLIENT`,
which is the recommended method and will select the highest TLS
version supported by both client and server.

Signed-off-by: Milas Bowman <[email protected]>
@milas milas requested a review from a team July 26, 2022 17:00
@milas milas self-assigned this Jul 26, 2022
@milas milas merged commit 56dd6de into docker:master Jul 26, 2022
@milas milas deleted the py-tls-deprecation branch July 26, 2022 19:12
@milas milas added this to the 6.0.0 milestone Jul 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants