Merge pull request #76 from chef/jfm/update

chef · Mar 4, 2021 · 77cf512 · 77cf512
2 parents 4240953 + 7621549
commit 77cf512
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/lib/win32/certstore/mixin/helper.rb b/lib/win32/certstore/mixin/helper.rb
@@ -22,7 +22,7 @@ class Certstore
     module Mixin
       module Helper
         # PSCommand to search certificate from thumbprint and either turn it into a pem or return a path to a pfx object
-        def cert_ps_cmd(thumbprint, store_location: "LocalMachine", export_password: "1234", output_path:)
+        def cert_ps_cmd(thumbprint, store_location: "LocalMachine", export_password: "1234", output_path: "")
           <<-EOH
             $cert = Get-ChildItem Cert:\'#{store_location}' -Recurse | Where { $_.Thumbprint -eq '#{thumbprint}' }
 
@@ -34,10 +34,8 @@ def cert_ps_cmd(thumbprint, store_location: "LocalMachine", export_password: "12
                 if($use.FriendlyName -like "Client Authentication" ){
                     return $true
                 }
-                else {
-                    return $false
-                }
               }
+              return $false
             }
 
             $result = test_cert_values