Merge pull request #16 from casselc/multiple-domain-fix

Fix bug with multiple domains
PKISharp · Oct 15, 2018 · d619d34 · d619d34
2 parents 88be279 + f4d28b9
commit d619d34
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/ACME-PS/internal/classes/crypto/Certificate.ps1 b/ACME-PS/internal/classes/crypto/Certificate.ps1
@@ -28,7 +28,7 @@ class Certificate {
 
         $sanBuilder = [System.Security.Cryptography.X509Certificates.SubjectAlternativeNameBuilder]::new();
         foreach ($dnsName in $dnsNames) {
-            $sanBuilder.AddDnsName($dnsNames);
+            $sanBuilder.AddDnsName($dnsName);
         }
 
         $distinguishedName = [X500DistinguishedName]::new("CN=$($dnsNames[0])");
@@ -51,4 +51,4 @@ class Certificate {
         $certRequest.CertificateExtensions.Add($sanBuilder.Build());
         return $certRequest.CreateSigningRequest();
     }
-}
+}