Merge pull request #17 from Keyfactor/optionalsubjectchecks

Allow optional department
Keyfactor · Jun 14, 2022 · 6d5a88b · 6d5a88b
2 parents e1a9129 + d3bf96e
commit 6d5a88b
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/SectigoCAProxy/SectigoCAProxy.cs b/src/SectigoCAProxy/SectigoCAProxy.cs
@@ -254,8 +254,12 @@ public override EnrollmentResult Enroll(ICertificateDataReader certificateDataRe
 
 				string ouStr = ParseSubject(subject, "OU=", false);
 
-				string department = productInfo.ProductParameters["Department"];
-				Logger.Trace($"Department: {department}");
+				string department = null;
+				if (productInfo.ProductParameters.ContainsKey("Department"))
+				{
+					department = productInfo.ProductParameters["Department"];
+					Logger.Trace($"Department: {department}");
+				}
 
 				var fieldList = Task.Run(async () => await Client.ListCustomFields()).Result;
 				var mandatoryFields = fieldList.CustomFields?.Where(f => f.mandatory);