Skip to content

Commit c396f66

Browse files
Merge pull request #19 from allantargino/master
simplifying certificate chain handling
2 parents 720b7bd + d1fde6d commit c396f66

File tree

1 file changed

+4
-5
lines changed

1 file changed

+4
-5
lines changed

src/Kubernetes.Auth.cs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -105,13 +105,12 @@ private bool CertificateValidationCallBack(
105105
// If there are errors in the certificate chain, look at each error to determine the cause.
106106
if ((sslPolicyErrors & SslPolicyErrors.RemoteCertificateChainErrors) != 0)
107107
{
108-
X509Chain chain0 = new X509Chain();
109-
chain0.ChainPolicy.RevocationMode = X509RevocationMode.NoCheck;
108+
chain.ChainPolicy.RevocationMode = X509RevocationMode.NoCheck;
110109

111110
// add all your extra certificate chain
112-
chain0.ChainPolicy.ExtraStore.Add(this.CaCert);
113-
chain0.ChainPolicy.VerificationFlags = X509VerificationFlags.AllowUnknownCertificateAuthority;
114-
var isValid = chain0.Build((X509Certificate2)certificate);
111+
chain.ChainPolicy.ExtraStore.Add(this.CaCert);
112+
chain.ChainPolicy.VerificationFlags = X509VerificationFlags.AllowUnknownCertificateAuthority;
113+
var isValid = chain.Build((X509Certificate2)certificate);
115114
return isValid;
116115
}
117116
else

0 commit comments

Comments
 (0)