Skip to content

Commit

Permalink
Added example of how ALPN should be working
Browse files Browse the repository at this point in the history
  • Loading branch information
TCROC committed May 8, 2023
1 parent 449624f commit 306952c
Show file tree
Hide file tree
Showing 5 changed files with 9 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@ public sealed class MqttClientOptionsBuilderTlsParameters
public Func<MqttClientCertificateValidationEventArgs, bool> CertificateValidationHandler { get; set; }

#if NET48 || NETCOREAPP3_1_OR_GREATER
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | SslProtocols.Tls13;
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | (SslProtocols)0x00003000 /*Tls13*/;
// public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | SslProtocols.Tls13;
#else
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | (SslProtocols)0x00003000 /*Tls13*/;
#endif
Expand Down
5 changes: 3 additions & 2 deletions Source/MQTTnet/Client/Options/MqttClientTlsOptions.cs
Original file line number Diff line number Diff line change
Expand Up @@ -32,11 +32,12 @@ public sealed class MqttClientTlsOptions
#if NETCOREAPP3_1_OR_GREATER
public List<System.Net.Security.SslApplicationProtocol> ApplicationProtocols { get; set; }

public System.Net.Security.CipherSuitesPolicy CipherSuitesPolicy { get; set; }
// public System.Net.Security.CipherSuitesPolicy CipherSuitesPolicy { get; set; }
#endif

#if NET48 || NETCOREAPP3_1_OR_GREATER
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | SslProtocols.Tls13;
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | (SslProtocols)0x00003000 /*Tls13*/;
// public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | SslProtocols.Tls13;
#else
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12 | (SslProtocols)0x00003000 /*Tls13*/;
#endif
Expand Down
2 changes: 1 addition & 1 deletion Source/MQTTnet/Implementations/MqttTcpChannel.cs
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ public async Task ConnectAsync(CancellationToken cancellationToken)
CertificateRevocationCheckMode =
_tcpOptions.TlsOptions.IgnoreCertificateRevocationErrors ? X509RevocationMode.NoCheck : _tcpOptions.TlsOptions.RevocationMode,
TargetHost = _tcpOptions.Server,
CipherSuitesPolicy = _tcpOptions.TlsOptions.CipherSuitesPolicy
// CipherSuitesPolicy = _tcpOptions.TlsOptions.CipherSuitesPolicy
};

await sslStream.AuthenticateAsClientAsync(sslOptions, cancellationToken).ConfigureAwait(false);
Expand Down
4 changes: 2 additions & 2 deletions Source/MQTTnet/Implementations/MqttTcpServerListener.cs
Original file line number Diff line number Diff line change
Expand Up @@ -202,8 +202,8 @@ async Task TryHandleClientConnectionAsync(CrossPlatformSocket clientSocket)
EnabledSslProtocols = _tlsOptions.SslProtocol,
CertificateRevocationCheckMode = _tlsOptions.CheckCertificateRevocation ? X509RevocationMode.Online : X509RevocationMode.NoCheck,
EncryptionPolicy = EncryptionPolicy.RequireEncryption,
CipherSuitesPolicy = _tlsOptions.CipherSuitesPolicy
}).ConfigureAwait(false);
// CipherSuitesPolicy = _tlsOptions.CipherSuitesPolicy
}, default).ConfigureAwait(false);
#else
await sslStream.AuthenticateAsServerAsync(
clientCertificate,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ public MqttServerTlsTcpEndpointOptions()
public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12;

#if NETCOREAPP3_1_OR_GREATER
public System.Net.Security.CipherSuitesPolicy CipherSuitesPolicy { get; set; }
// public System.Net.Security.CipherSuitesPolicy CipherSuitesPolicy { get; set; }
#endif
}
}

0 comments on commit 306952c

Please sign in to comment.