merge from main

Author: dgaley

CHANGELOG.md

@@ -23,4 +23,7 @@
 * Remove caching of product ID lookups from DigiCert account  
 
 ### 2.2.0  
-* Add support for duplicating certs
+* Add support for duplicating certs
+
+### 2.2.1  
+* Properly mark 'needs_approval' status as Pending rather than Failed  

digicert-certcentral-caplugin/CertCentralCAPlugin.cs

@@ -5,6 +5,7 @@
 using Keyfactor.Extensions.CAPlugin.DigiCert.Client;
 using Keyfactor.Extensions.CAPlugin.DigiCert.Models;
 using Keyfactor.Logging;
+using Keyfactor.PKI;
 using Keyfactor.PKI.Enums;
 using Keyfactor.PKI.Enums.EJBCA;
 
@@ -126,12 +127,15 @@ public async Task<EnrollmentResult> Enroll(string csr, string subject, Dictionar
 			string commonName = null, organization = null, orgUnit = null;
 			try
 			{
-				subjectParsed = new X509Name(subject);
+				subjectParsed = new X509Name(true, PKIConstants.X509.OIDLookup, subject);
 				commonName = subjectParsed.GetValueList(X509Name.CN).Cast<string>().LastOrDefault();
 				organization = subjectParsed.GetValueList(X509Name.O).Cast<string>().LastOrDefault();
 				orgUnit = subjectParsed.GetValueList(X509Name.OU).Cast<string>().LastOrDefault();
 			}
-			catch (Exception) { }
+			catch (Exception exc)
+			{
+				_logger.LogInformation($"Error while parsing subject. This might be expected. Error message: {exc.Message}");
+			}
 
 			if (commonName == null)
 			{