Skip to content

Commit

Permalink
enable ntlm tests on RHEL7 again (#85633)
Browse files Browse the repository at this point in the history
  • Loading branch information
wfurt authored May 2, 2023
1 parent c240ade commit 4ead807
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ public static bool IsNtlmInstalled()
{
// GSS on Linux does not work with OpenSSL 3.0. Fix was submitted to gss-ntlm but it will take a while to make to
// all supported distributions. The second part of the check should be removed when it does.
return Interop.NetSecurityNative.IsNtlmInstalled() && (!PlatformDetection.IsOpenSslSupported || PlatformDetection.OpenSslVersion.Major < 3) && !PlatformDetection.IsRedHatFamily7;
return Interop.NetSecurityNative.IsNtlmInstalled() && (!PlatformDetection.IsOpenSslSupported || PlatformDetection.OpenSslVersion.Major < 3);
}
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
using System.Security.Principal;
using System.Text;
using System.Threading.Tasks;
using Microsoft.DotNet.XUnitExtensions;
using Xunit;

namespace System.Net.Security.Tests
Expand Down Expand Up @@ -87,10 +86,6 @@ public void Package_Supported_NTLM()
[ConditionalFact(nameof(IsNtlmUnavailable))]
public void Package_Unsupported_NTLM()
{
if (PlatformDetection.IsRedHatFamily7)
{
throw new SkipTestException("https://github.com/dotnet/runtime/issues/83540");
}
NegotiateAuthenticationClientOptions clientOptions = new NegotiateAuthenticationClientOptions { Package = "NTLM", Credential = s_testCredentialRight, TargetName = "HTTP/foo" };
NegotiateAuthentication negotiateAuthentication = new NegotiateAuthentication(clientOptions);
NegotiateAuthenticationStatusCode statusCode;
Expand Down

0 comments on commit 4ead807

Please sign in to comment.