Respect the CheckCertificateRevocation property.

Author: JoeShook

MimeKit/Cryptography/BouncyCastleSecureMimeContext.cs

@@ -694,11 +694,12 @@ protected IList<X509Certificate> BuildCertificateChain (X509Certificate certific
 
 			var parameters = new PkixBuilderParameters (GetTrustedAnchors (), selector) {
 				ValidityModel = PkixParameters.PkixValidityModel,
-				IsRevocationEnabled = false,
+				IsRevocationEnabled = CheckCertificateRevocation,
 				Date = DateTime.UtcNow
 			};
 			parameters.AddStoreCert (intermediates);
 			parameters.AddStoreCert (GetIntermediateCertificates ());
+			parameters.AddStoreCrl (GetCertificateRevocationLists ());
 
 			var builder = new PkixCertPathBuilder ();
 			var result = builder.Build (parameters);
@@ -725,7 +726,7 @@ PkixCertPath BuildCertPath (ISet<TrustAnchor> anchors, IStore<X509Certificate> c
 
 			var parameters = new PkixBuilderParameters (anchors, selector) {
 				ValidityModel = PkixParameters.PkixValidityModel,
-				IsRevocationEnabled = false
+				IsRevocationEnabled = CheckCertificateRevocation
 			};
 			parameters.AddStoreCert (intermediates);
 			parameters.AddStoreCrl (crls);

UnitTests/Cryptography/ApplicationPkcs7MimeTests.cs

@@ -927,7 +927,7 @@ public async Task TestSignAndEncryptDnsNamesAsync ()
 	[TestFixture]
 	public class ApplicationPkcs7MimeTests : ApplicationPkcs7MimeTestsBase
 	{
-		readonly TemporarySecureMimeContext ctx = new TemporarySecureMimeContext (new SecureRandom (new CryptoApiRandomGenerator ())) { CheckCertificateRevocation = true };
+		readonly TemporarySecureMimeContext ctx = new TemporarySecureMimeContext (new SecureRandom (new CryptoApiRandomGenerator ())) { CheckCertificateRevocation = false };
 
 		public ApplicationPkcs7MimeTests ()
 		{
@@ -947,7 +947,7 @@ class MySecureMimeContext : DefaultSecureMimeContext
 		{
 			public MySecureMimeContext () : base ("pkcs7.db", "no.secret")
 			{
-				CheckCertificateRevocation = true;
+				CheckCertificateRevocation = false;
 			}
 		}
 

UnitTests/Cryptography/SecureMimeTests.cs

@@ -2734,7 +2734,7 @@ public async Task TestSecureMimeVerifyMixedLineEndingsAsync ()
 	[TestFixture]
 	public class SecureMimeTests : SecureMimeTestsBase
 	{
-		readonly TemporarySecureMimeContext ctx = new TemporarySecureMimeContext (new SecureRandom (new CryptoApiRandomGenerator ())) { CheckCertificateRevocation = true };
+		readonly TemporarySecureMimeContext ctx = new TemporarySecureMimeContext (new SecureRandom (new CryptoApiRandomGenerator ())) { CheckCertificateRevocation = false };
 
 		protected override SecureMimeContext CreateContext ()
 		{
@@ -2749,7 +2749,7 @@ class MySecureMimeContext : DefaultSecureMimeContext
 		{
 			public MySecureMimeContext () : base ("smime.db", "no.secret")
 			{
-				CheckCertificateRevocation = true;
+				CheckCertificateRevocation = false;
 			}
 		}