Merge pull request #552 from techraf/fixing_matching_certificate

Removed piping results to shasum to avoid test passing incorrectly
mizzy · Jan 15, 2016 · 3a76cc1 · 3a76cc1
2 parents 069e104 + 48c0bda
commit 3a76cc1
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/serverspec/type/x509_private_key.rb b/lib/serverspec/type/x509_private_key.rb
@@ -12,9 +12,8 @@ def encrypted?
     end
 
     def has_matching_certificate?(cert_file)
-      mac_op = "openssl sha -sha512"
-      h1 = @runner.run_command("openssl x509 -noout -modulus -in #{cert_file} | #{mac_op}")
-      h2 = @runner.run_command("openssl rsa -noout -modulus -in #{name} | #{mac_op}")
+      h1 = @runner.run_command("openssl x509 -noout -modulus -in #{cert_file}")
+      h2 = @runner.run_command("openssl rsa -noout -modulus -in #{name}")
       (h1.stdout == h2.stdout) && (h1.exit_status == 0) && (h2.exit_status == 0)
     end
   end