From c778cd7a0f8844fe71ebe3388231131b9e3ff2fa Mon Sep 17 00:00:00 2001 From: Jay424 Date: Wed, 28 Oct 2020 17:14:09 +0100 Subject: [PATCH 1/2] Fixed evaluation of IgnoreCertificateRevocationErrors to correctly set SslClientAuthenticationOptions.CertificateRevocationCheckMode The if was just flipped. --- Source/MQTTnet/Implementations/MqttTcpChannel.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Source/MQTTnet/Implementations/MqttTcpChannel.cs b/Source/MQTTnet/Implementations/MqttTcpChannel.cs index 5d96740..8a349ec 100644 --- a/Source/MQTTnet/Implementations/MqttTcpChannel.cs +++ b/Source/MQTTnet/Implementations/MqttTcpChannel.cs @@ -87,7 +87,7 @@ namespace MQTTnet.Implementations ApplicationProtocols = _options.TlsOptions.ApplicationProtocols, ClientCertificates = LoadCertificates(), EnabledSslProtocols = _options.TlsOptions.SslProtocol, - CertificateRevocationCheckMode = _options.TlsOptions.IgnoreCertificateRevocationErrors ? X509RevocationMode.Online : X509RevocationMode.NoCheck, + CertificateRevocationCheckMode = _options.TlsOptions.IgnoreCertificateRevocationErrors ? X509RevocationMode.NoCheck : X509RevocationMode.Online, TargetHost = _options.Server }; From 3ea281ff849c0f79461a678aa2f7fe210f95664d Mon Sep 17 00:00:00 2001 From: Jay424 Date: Wed, 28 Oct 2020 17:58:48 +0100 Subject: [PATCH 2/2] Updated release notes --- Build/MQTTnet.nuspec | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/Build/MQTTnet.nuspec b/Build/MQTTnet.nuspec index 1602858..2bf6cb7 100644 --- a/Build/MQTTnet.nuspec +++ b/Build/MQTTnet.nuspec @@ -14,7 +14,8 @@ * [Client] Fixed wrong value for "ClientWasConnected" in "MqttClientDisconnectedEventArgs" #976 (thanks to @dbeinder). * [Client] Added direct support for Amazon AWS connections (requires .NET Core 3.1) (thanks to @henning-krause). -* [Client] handle disconnect package and propagate disconnect reason code (thanks to @JanEggers) +* [Client] handle disconnect package and propagate disconnect reason code (thanks to @JanEggers) +* [Client] Fixed an issue when checking for revoked SSL certificates for target platform NETCOREAPP3_1. * [ManagedClient] Exposed the internal MQTT client. * [Server] Added client message queue interceptor for QoS level (thanks to @msallin). * [Server] improved behavior when multiple connections fight over a session (thanks to @JanEggers)