Fix Remote Deletion of SSL Certificates

get_remote_certificate doesnt need ssl data passed anymore, this makes sure that no more calls pass this argument (Closes #62)
ministryofjustice · Jun 1, 2015 · 6017572 · 6017572
1 parent 30f746c
commit 6017572
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/bootstrap_cfn/iam.py b/bootstrap_cfn/iam.py
@@ -256,8 +256,7 @@ def upload_certificate(self, cert_name, stack_name, ssl_data, force=False):
 
         try:
             if force or not self.get_remote_certificate(cert_name,
-                                                        stack_name,
-                                                        ssl_data):
+                                                        stack_name):
                 self.conn_iam.upload_server_cert(cert_id, cert_body,
                                                  private_key,
                                                  cert_chain)
@@ -300,8 +299,7 @@ def delete_certificate(self, cert_name, stack_name, ssl_data):
         # continue to delete other certs
         try:
             if self.get_remote_certificate(cert_name,
-                                           stack_name,
-                                           ssl_data):
+                                           stack_name):
                 self.conn_iam.delete_server_cert(cert_id)
                 logging.info("IAM::delete_certificate: "
                              "Deleting certificate '%s'.."