code review feedback

Author: davidkelly

docs/getting-started.md

@@ -65,8 +65,7 @@ var cluster = Cluster.Create(
 ```
 
 > [!NOTE]
-> The client certificate must contain a private key. When using mTLS, no HTTP `Authorization` header
-> is sent — authentication is handled entirely during the TLS handshake.
+> The client certificate must contain a private key.
 
 #### Updating Credentials
 

src/Couchbase.Analytics/Cluster.cs

@@ -125,10 +125,6 @@ public static Cluster Create(ICredential credential, ClusterOptions clusterOptio
         return new Cluster(credential, clusterOptions);
     }
 
-    // ── Binary-compatible forwarding overloads ──────────────────────────
-    // These preserve the exact method signatures from before the ICredential
-    // widening, so that existing compiled code continues to resolve correctly
-    // at runtime without recompilation.
 
     /// <summary>
     /// Creates a cluster with a connection string, username/password credential, and an options builder.

src/Couchbase.Analytics/HTTP/CertificateCredential.cs

@@ -96,7 +96,7 @@ public static CertificateCredential FromPem(string certPath, string keyPath) =>
     /// </summary>
     private bool PrintMembers(StringBuilder builder)
     {
-        builder.Append($"Subject = {Certificate.Subject}, Thumbprint = {Certificate.Thumbprint}");
+        builder.Append($"{nameof(Certificate.Subject)} = {Certificate.Subject}, {nameof(Certificate.Thumbprint)} = {Certificate.Thumbprint}");
         return true;
     }
 }

src/Couchbase.Analytics/HTTP/Credential.cs

@@ -52,7 +52,7 @@ public static Credential Create(string username, string password)
     /// </summary>
     protected virtual bool PrintMembers(System.Text.StringBuilder builder)
     {
-        builder.Append($"Username = {Username}");
+        builder.Append($"{nameof(Username)} = {Username}");
         return true;
     }
 }

src/Couchbase.Analytics/HTTP/JwtCredential.cs

@@ -49,7 +49,7 @@ public static JwtCredential Create(string token)
     /// </summary>
     private bool PrintMembers(System.Text.StringBuilder builder)
     {
-        builder.Append($"Token = <{Token.Length} chars>");
+        builder.Append($"{nameof(Token)} = <{Token.Length} chars>");
         return true;
     }
 }

src/Couchbase.Analytics/Internal/HTTP/CouchbaseHttpClientFactory.cs

@@ -192,9 +192,7 @@ public HttpClient Create()
     /// The old handler is disposed after a grace period via <see cref="DisposeAfterDelayAsync"/>.
     /// This follows the same pattern as <c>IHttpClientFactory</c> in ASP.NET Core, which defers
     /// handler disposal to allow request draining. The grace period is generous (1 minute) to
-    /// accommodate the maximum query timeout. This avoids the <see cref="ObjectDisposedException"/>
-    /// race that would occur with immediate disposal (a known issue in the operational SDK's
-    /// <c>CouchbaseHttpClientFactory.RecreateHandler</c>).
+    /// accommodate the maximum query timeout.
     /// </para>
     /// </remarks>
     /// <param name="newCredential">The new credential that triggered the rebuild.</param>