Small corrections.

Author: papafe

src/MongoDB.Driver/Core/Configuration/ClusterBuilder.cs

@@ -162,7 +162,7 @@ public ClusterBuilder ConfigureTcp(Func<TcpStreamSettings, TcpStreamSettings> co
         /// Configures the SOCKS5 proxy settings for the cluster.
         /// </summary>
         /// <param name="configurator"> The SOCKS5 proxy settings configurator delegate.</param>
-        /// <returns> A reconfigured cluster builder.</returns>
+        /// <returns>A reconfigured cluster builder.</returns>
         public ClusterBuilder ConfigureSocks5Proxy(Func<Socks5ProxyStreamSettings, Socks5ProxyStreamSettings> configurator)
         {
             Ensure.IsNotNull(configurator, nameof(configurator));

src/MongoDB.Driver/Core/Configuration/Socks5ProxyStreamSettings.cs

@@ -20,7 +20,7 @@ namespace MongoDB.Driver.Core.Configuration;
 /// <summary>
 /// Represents settings for a SOCKS5 proxy stream.
 /// </summary>
-public class Socks5ProxyStreamSettings
+public sealed class Socks5ProxyStreamSettings
 {
     private Socks5ProxySettings _socks5ProxySettings;
 
@@ -44,7 +44,5 @@ public Socks5ProxyStreamSettings(Optional<Socks5ProxySettings> socks5ProxySettin
     /// <param name="socks5ProxySettings"></param>
     /// <returns></returns>
     public Socks5ProxyStreamSettings With(Socks5ProxySettings socks5ProxySettings)
-    {
-        return new Socks5ProxyStreamSettings(socks5ProxySettings);
-    }
+        => new(socks5ProxySettings);
 }

src/MongoDB.Driver/Core/Connections/Socks5Helper.cs

@@ -145,7 +145,7 @@ public static async Task PerformSocks5HandshakeAsync(Stream stream, EndPoint end
             await stream.ReadBytesAsync(buffer, 0, 2, cancellationToken).ConfigureAwait(false);
             var requiresAuthenticationStep = ProcessGreetingResponse(buffer, useAuth);
 
-            // If we have username and password, but the proxy doesn't need them, we skip.
+            // If we have username and password, but the proxy doesn't need them, we skip the authentication step.
             if (requiresAuthenticationStep)
             {
                 var authenticationRequestLength = CreateAuthenticationRequest(buffer, authenticationSettings);
@@ -289,7 +289,7 @@ private static int ProcessConnectResponse(byte[] buffer)
             AddressTypeIPv4 => 5,
             AddressTypeIPv6 => 17,
             AddressTypeDomain => buffer[4] + 2,
-            _ => throw new IOException("Unknown address type in SOCKS5 reply.")
+            _ => throw new IOException($"Unknown address type in SOCKS5 reply: {buffer[3]}.")
         };
     }
 

src/MongoDB.Driver/MongoUrlBuilder.cs

@@ -448,7 +448,7 @@ public string Password
         }
 
         /// <summary>
-        ///
+        /// Gets or sets the proxy host.
         /// </summary>
         public string ProxyHost
         {
@@ -460,7 +460,7 @@ public string ProxyHost
         }
 
         /// <summary>
-        ///
+        /// Gets or sets the proxy port.
         /// </summary>
         /// <exception cref="ArgumentOutOfRangeException"></exception>
         public int? ProxyPort
@@ -473,7 +473,7 @@ public int? ProxyPort
         }
 
         /// <summary>
-        ///
+        /// Gets or sets the proxy username.
         /// </summary>
         public string ProxyUsername
         {
@@ -485,7 +485,7 @@ public string ProxyUsername
         }
 
         /// <summary>
-        ///
+        /// Gets or sets the proxy password.
         /// </summary>
         public string ProxyPassword
         {

tests/MongoDB.Driver.Tests/Specifications/socks5-support/Socks5SupportProseTests.cs

@@ -75,6 +75,7 @@ public static IEnumerable<object[]> GetTestCombinations()
             });
     }
 
+    //Prose test: https://github.com/mongodb/specifications/blob/a6dbd208462d97f97c813560cac5cf25925bb0cf/source/socks5-support/tests/README.md
     [Theory]
     [MemberData(nameof(GetTestCombinations))]
     public async Task TestConnectionStrings(string id, string connectionString, bool expectedResult, bool useTls, bool async)