Merge pull request stratisproject#220 from quantumagi/dltremredun

Remove superfluous condition in IssueCertificate

src/CertificateAuthority/CaCertificatesManager.cs

@@ -213,7 +213,7 @@ private CertificateInfoModel IssueCertificate(Pkcs10CertificationRequest certReq
             CertificateInfoModel existingCertificate = this.repository.GetCertificateIssuedByAccountId(cred);
 
             // TODO: Should this actually throw instead?
-            if (existingCertificate != null && existingCertificate.Status != CertificateStatus.Revoked)
+            if (existingCertificate != null)
                 return existingCertificate;
 
             string knownSubjectDistinguishedName = this.GetClientCertificateSubjectDistinguishedName(new CredentialsAccessModel(model.AccountId, model.Password, AccountAccessFlags.BasicAccess));
@@ -669,6 +669,10 @@ public bool RevokeCertificateForAccount(CredentialsAccessWithModel<CredentialsMo
             {
                 CertificateInfoModel certificate = this.repository.GetCertificateIssuedByAccountId(model);
 
+                // TODO: Should this actually throw instead?
+                if (certificate == null)
+                    return true;
+
                 this.repository.CertStatusesByThumbprint[certificate.Thumbprint] = CertificateStatus.Revoked;
 
                 CertificateInfoModel certToEdit = dbContext.Certificates.Single(x => x.Thumbprint == certificate.Thumbprint);