Apply suggestions from code review

Co-authored-by: David Engel <[email protected]>
dotnet · Jul 30, 2021 · abc1649 · abc1649
1 parent dcaab15
commit abc1649
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/BUILDGUIDE.md b/BUILDGUIDE.md
@@ -259,7 +259,7 @@ To use this feature, you must enable the following AppContext switch at applicat
 
 ## Enabling OS secure protocols preference
 
-Tls 1.3 has been taken out in view of the fact that the driver lacks full support. To jump back on the OS preferences as same as earlier, you must enable the following AppContext switch on application's startup:
+TLS 1.3 has been excluded due to the fact that the driver lacks full support. To enable OS preferences as before, enable the following AppContext switch on application startup:
 
 **"Switch.Microsoft.Data.SqlClient.EnableSecureProtocolsByOS"**
 

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
@@ -13,7 +13,7 @@ namespace Microsoft.Data.SqlClient.SNI
     internal abstract class SNIHandle
     {
         /// <summary>
-        /// Exclude Tls 1.3 (not fully supported).
+        /// Exclude TLS 1.3 (not fully supported).
         /// </summary>
         protected readonly SslProtocols SupportedProtocols = LocalAppContextSwitches.UseSystemDefaultSecureProtocols ? SslProtocols.None : SslProtocols.Tls12 | SslProtocols.Tls11 | SslProtocols.Tls
         //protected readonly SslProtocols SupportedProtocols = SslProtocols.Tls12 | SslProtocols.Tls11 | SslProtocols.Tls