8000 SslProtocols update by tdykstra · Pull Request #4410 · dotnet/dotnet-api-docs · GitHub
[go: up one dir, main page]

Skip to content

SslProtocols update #4410

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 26, 2020
Merged

SslProtocols update #4410

merged 1 commit into from
Jun 26, 2020

Conversation

tdykstra
Copy link
Contributor

Fixes dotnet/docs#10201

Per the issue:

  • SslProtocols.Default should not be used; it should be replaced with SslProtocols.None.

    No instances using SslProtocols.Default were found.

  • The documentation for Tls should be added to note that it's provided for backward compatibility only.

    Added note to SslProtocols.Tls.

  • SslProtocols.Ssl2, SslProtocols.Ssl3, and SslProtocols.Tls are all obsolete. Use the OS default (None).

    Found instances of SslProtocols.Ssl3 and SslProtocols.Tls in snippet files, changed them to SslProtocols.None -- but the Snippet7 that they occur in isn't actually used anywhere.

@dotnet-bot dotnet-bot added this to the June 2020 milestone Jun 18, 2020
@tdykstra tdykstra merged commit 44f5723 into dotnet:master Jun 26, 2020
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.

**Check SSLProtocols enum usage**
4 participants
0