Fixed a couple minor bugs related to switching profiles when creating new orders that match existing orders.

Author: rmbolger

Posh-ACME/Public/New-PACertificate.ps1

@@ -128,6 +128,7 @@ function New-PACertificate {
     # - has different SANs
     # - has different CSR
     # - has different Lifetime
+    # - has different Profile
     $order = Get-PAOrder -Name $Name -Refresh
     $oldOrder = $null
     $SANs = @($Domain | Where-Object { $_ -ne $Domain[0] }) | Sort-Object
@@ -138,7 +139,8 @@ function New-PACertificate {
         ('CertKeyLength' -in $psbKeys -and $CertKeyLength -ne $order.KeyLength) -or
         ($SANs -join ',') -ne (($order.SANs | Sort-Object) -join ',') -or
         ($csrDetails -and $csrDetails.Base64Url -ne $order.CSRBase64Url ) -or
-        ($LifetimeDays -and $LifetimeDays -ne $order.LifetimeDays) )
+        ($LifetimeDays -and $LifetimeDays -ne $order.LifetimeDays) -or
+        ($Profile -and $Profile -ne $order.Profile) )
     {
 
         $oldOrder = $order

Posh-ACME/Public/New-PAOrder.ps1

@@ -125,10 +125,11 @@ function New-PAOrder {
 
         $oldDomains = (@($order.MainDomain) + @($order.SANs) | Sort-Object) -join ','
 
-        # skip confirmation if the Domains or KeyLength are different regardless
-        # of the original order status or if the order is pending but expired
+        # skip confirmation if the Domains, KeyLength, or Profile are different
+        # regardless of the original order status or if the order is pending but expired
         if ( ($order -and ($KeyLength -ne $order.KeyLength -or
              ($oldDomains -ne ($Domain | Sort-Object) -join ',') -or
+             ($Profile -and $Profile -ne $order.Profile) -or
              ($order.status -eq 'pending' -and (Get-DateTimeOffsetNow) -gt ([DateTimeOffset]::Parse($order.expires))) ))) {
             # do nothing