From: Ralf Jung <post@ralfj.de>
Date: Fri, 7 Apr 2017 09:07:47 +0000 (+0200)
Subject: try to clean up temporary CSR file if ACME-tiny fails
X-Git-Url: https://git.ralfj.de/lets-encrypt-tiny.git/commitdiff_plain/ced309aad271bf75269e224edc5f92ff7868187d?ds=sidebyside;hp=0a7958cbbe3105bbc8d079717af607b436804cbf

try to clean up temporary CSR file if ACME-tiny fails
---

diff --git a/letsencrypt-tiny b/letsencrypt-tiny
index 484d99d..78d38b7 100755
--- a/letsencrypt-tiny
+++ b/letsencrypt-tiny
@@ -61,15 +61,17 @@ def acme(keyfilename, certfilename, domains):
     assert not os.path.exists(csrfilename), "The temporary CSR file {} still exists. It seems something went wrong on a previous request. You may want to remove the file manually.".format(csrfilename)
     with open(csrfilename, 'wb') as file:
         file.write(csr)
-    # call acme-tiny as a script
-    acme_tiny = os.path.join(config['acme']['acme-tiny'], 'acme_tiny.py')
-    signed_crt = subprocess.check_output(["python", acme_tiny, "--quiet", "--account-key", accountkey, "--csr", csrfilename, "--acme-dir", config['acme']['challenge-dir']])
-    # save new certificate
-    make_backup(certfilename)
-    with open(certfilename, 'wb') as f:
-        f.write(signed_crt)
-    # clean up
-    os.remove(csrfilename)
+    try:
+        # call acme-tiny as a script
+        acme_tiny = os.path.join(config['acme']['acme-tiny'], 'acme_tiny.py')
+        signed_crt = subprocess.check_output(["python", acme_tiny, "--quiet", "--account-key", accountkey, "--csr", csrfilename, "--acme-dir", config['acme']['challenge-dir']])
+        # save new certificate
+        make_backup(certfilename)
+        with open(certfilename, 'wb') as f:
+            f.write(signed_crt)
+    finally:
+        # clean up
+        os.remove(csrfilename)
 
 def openssl_genrsa(keyfilename):
     with subprocess.Popen(["openssl", "genrsa", str(int(config['DEFAULT']['key-length']))], stdout=subprocess.PIPE, stderr=subprocess.PIPE) as f: