diff --git a/Source/MQTTnet.AspnetCore/MQTTnet.AspNetCore.csproj b/Source/MQTTnet.AspnetCore/MQTTnet.AspNetCore.csproj index dc44208d..3c9cd228 100644 --- a/Source/MQTTnet.AspnetCore/MQTTnet.AspNetCore.csproj +++ b/Source/MQTTnet.AspnetCore/MQTTnet.AspNetCore.csproj @@ -27,7 +27,6 @@ - diff --git a/Source/MQTTnet/Client/Options/MqttClientOptionsBuilder.cs b/Source/MQTTnet/Client/Options/MqttClientOptionsBuilder.cs index e3a3afe7..68d1aa3f 100644 --- a/Source/MQTTnet/Client/Options/MqttClientOptionsBuilder.cs +++ b/Source/MQTTnet/Client/Options/MqttClientOptionsBuilder.cs @@ -281,7 +281,7 @@ namespace MQTTnet.Client.Options CertificateValidationCallback = _tlsParameters.CertificateValidationCallback, #pragma warning restore CS0618 // Type or member is obsolete #if NETCOREAPP3_1 - ApplicationProtocols = _tlsParameters.ApplicationProtocols, + ApplicationProtocols = _tlsParameters.ApplicationProtocols, #endif CertificateValidationHandler = _tlsParameters.CertificateValidationHandler, IgnoreCertificateChainErrors = _tlsParameters.IgnoreCertificateChainErrors, diff --git a/Source/MQTTnet/Client/Options/MqttClientTlsOptions.cs b/Source/MQTTnet/Client/Options/MqttClientTlsOptions.cs index 1675ded7..106aeb14 100644 --- a/Source/MQTTnet/Client/Options/MqttClientTlsOptions.cs +++ b/Source/MQTTnet/Client/Options/MqttClientTlsOptions.cs @@ -23,7 +23,7 @@ namespace MQTTnet.Client.Options #endif #if NETCOREAPP3_1 - public List ApplicationProtocols {get;set;} + public List ApplicationProtocols { get; set; } #endif public SslProtocols SslProtocol { get; set; } = SslProtocols.Tls12; diff --git a/Source/MQTTnet/Implementations/MqttTcpChannel.cs b/Source/MQTTnet/Implementations/MqttTcpChannel.cs index 312d7046..5d96740d 100644 --- a/Source/MQTTnet/Implementations/MqttTcpChannel.cs +++ b/Source/MQTTnet/Implementations/MqttTcpChannel.cs @@ -81,26 +81,24 @@ namespace MQTTnet.Implementations var sslStream = new SslStream(networkStream, false, InternalUserCertificateValidationCallback); try { - #if NETCOREAPP3_1 - var sslOptions = new SslClientAuthenticationOptions - { - ApplicationProtocols = _options.TlsOptions.ApplicationProtocols, - ClientCertificates = LoadCertificates(), - EnabledSslProtocols = _options.TlsOptions.SslProtocol, - CertificateRevocationCheckMode = _options.TlsOptions.IgnoreCertificateRevocationErrors ? X509RevocationMode.Online : X509RevocationMode.NoCheck, - TargetHost = _options.Server - }; - await sslStream.AuthenticateAsClientAsync(sslOptions, cancellationToken).ConfigureAwait(false); + var sslOptions = new SslClientAuthenticationOptions + { + ApplicationProtocols = _options.TlsOptions.ApplicationProtocols, + ClientCertificates = LoadCertificates(), + EnabledSslProtocols = _options.TlsOptions.SslProtocol, + CertificateRevocationCheckMode = _options.TlsOptions.IgnoreCertificateRevocationErrors ? X509RevocationMode.Online : X509RevocationMode.NoCheck, + TargetHost = _options.Server + }; + + await sslStream.AuthenticateAsClientAsync(sslOptions, cancellationToken).ConfigureAwait(false); #else - - - await sslStream.AuthenticateAsClientAsync(_options.Server, LoadCertificates(), _options.TlsOptions.SslProtocol, !_options.TlsOptions.IgnoreCertificateRevocationErrors).ConfigureAwait(false); + await sslStream.AuthenticateAsClientAsync(_options.Server, LoadCertificates(), _options.TlsOptions.SslProtocol, !_options.TlsOptions.IgnoreCertificateRevocationErrors).ConfigureAwait(false); #endif } catch { -#if NETSTANDARD2_1 +#if NETSTANDARD2_1 || NETCOREAPP3_1 await sslStream.DisposeAsync().ConfigureAwait(false); #else sslStream.Dispose(); @@ -224,7 +222,7 @@ namespace MQTTnet.Implementations bool InternalUserCertificateValidationCallback(object sender, X509Certificate x509Certificate, X509Chain chain, SslPolicyErrors sslPolicyErrors) { -#region OBSOLETE + #region OBSOLETE #pragma warning disable CS0618 // Type or member is obsolete var certificateValidationCallback = _options?.TlsOptions?.CertificateValidationCallback; @@ -233,7 +231,7 @@ namespace MQTTnet.Implementations { return certificateValidationCallback(x509Certificate, chain, sslPolicyErrors, _clientOptions); } -#endregion + #endregion var certificateValidationHandler = _options?.TlsOptions?.CertificateValidationHandler; if (certificateValidationHandler != null)