Change | Using default system protocols for TDS8 on Managed code (#1678)

dotnet · Jul 26, 2022 · 13b8c7d · 13b8c7d
1 parent 620cf46
commit 13b8c7d
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SNI/SNIHandle.cs b/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SNI/SNIHandle.cs
@@ -18,7 +18,7 @@ namespace Microsoft.Data.SqlClient.SNI
     internal abstract class SNIHandle
     {
         /// <summary>
-        /// Exclude TLS 1.3 (not fully supported).
+        /// Exclude TLS 1.3 in TLS-over-TDS modes (TDS 7.4 and below)
         /// </summary>
         protected static readonly SslProtocols s_supportedProtocols = LocalAppContextSwitches.UseSystemDefaultSecureProtocols ? SslProtocols.None : SslProtocols.Tls12 | SslProtocols.Tls11 | SslProtocols.Tls
         //protected readonly SslProtocols SupportedProtocols = SslProtocols.Tls12 | SslProtocols.Tls11 | SslProtocols.Tls
@@ -36,7 +36,6 @@ protected static async Task AuthenticateAsClientAsync(SslStream sslStream, strin
             {
                 TargetHost = serverNameIndication,
                 ApplicationProtocols = s_tdsProtocols,
-                EnabledSslProtocols = s_supportedProtocols,
                 ClientCertificates = certificate
             };
             await sslStream.AuthenticateAsClientAsync(sslClientOptions, token);