Skip to content

Commit 25d2636

Browse files
Use assembly visibility here
Per @bording suggestion. Plus some drive by cosmetic comment changes.
1 parent 8398be5 commit 25d2636

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

projects/client/RabbitMQ.Client/src/client/api/SslOption.cs

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ public SslOption()
7575
}
7676

7777
/// <summary>
78-
/// Retrieve or set the set of ssl policy errors that are deemed acceptable.
78+
/// Retrieve or set the set of TLS policy errors that are deemed acceptable.
7979
/// </summary>
8080
public SslPolicyErrors AcceptablePolicyErrors { get; set; }
8181

@@ -90,13 +90,13 @@ public SslOption()
9090
public string CertPath { get; set; }
9191

9292
/// <summary>
93-
/// An optional client specified SSL certificate selection callback. If this is not specified,
93+
/// An optional client specified TLS certificate selection callback. If this is not specified,
9494
/// the first valid certificate found will be used.
9595
/// </summary>
9696
public LocalCertificateSelectionCallback CertificateSelectionCallback { get; set; }
9797

9898
/// <summary>
99-
/// An optional client specified SSL certificate validation callback. If this is not specified,
99+
/// An optional client specified TLS certificate validation callback. If this is not specified,
100100
/// the default callback will be used in conjunction with the <see cref="AcceptablePolicyErrors"/> property to
101101
/// determine if the remote server certificate is valid.
102102
/// </summary>
@@ -135,13 +135,13 @@ public X509CertificateCollection Certs
135135
public bool CheckCertificateRevocation { get; set; }
136136

137137
/// <summary>
138-
/// Flag specifying if Ssl should indeed be used.
138+
/// Flag specifying if TLS should indeed be used.
139139
/// </summary>
140140
public bool Enabled { get; set; }
141141

142142
/// <summary>
143143
/// Retrieve or set server's Canonical Name.
144-
/// This MUST match the CN on the Certificate else the SSL connection will fail.
144+
/// This MUST match the Subject Alternative Name or CN on the Certificate else the TLS connection will fail.
145145
/// </summary>
146146
public string ServerName { get; set; }
147147

@@ -155,7 +155,7 @@ public X509CertificateCollection Certs
155155
/// Only used in environments where System.Security.Authentication.SslProtocols.None
156156
/// is unavailable or effectively disabled, as reported by System.Net.ServicePointManager.
157157
/// </summary>
158-
public SslProtocols UseFallbackTlsVersions()
158+
internal SslProtocols UseFallbackTlsVersions()
159159
{
160160
this.Version = SslProtocols.Tls12;
161161
return Version;

0 commit comments

Comments
 (0)