do not keep CSRs

[lets-encrypt-tiny.git] / letsencrypt-tiny

diff --git a/letsencrypt-tiny b/letsencrypt-tiny
index 57a2e60269ffab56dde4ae2986399c70b251f29b..3b89db7b9d2dcec7d7d1230ced46727a0e942a39 100755 (executable)
--- a/letsencrypt-tiny
+++ b/letsencrypt-tiny
@@ -13,7 +13,7 @@ def readConfig(fname, defSection = 'DEFAULT'):
 
 def certfile(name):
     global config
-    return os.path.join(config['dirs']['certs'], name + ".crt" + ('' if suff is None else '+'+suff) )
+    return os.path.join(config['dirs']['certs'], name + ".crt" )
 
 def keyfile(name):
     global config
@@ -21,7 +21,7 @@ def keyfile(name):
 
 def csrfile(name):
     global config
-    return os.path.join(config['dirs']['csrs'], name + ".csr")
+    return os.path.join(config['dirs']['keys'], name + ".csr.tmp")
 
 def make_backup(fname):
     if os.path.exists(fname):
@@ -30,7 +30,7 @@ def make_backup(fname):
         while True:
             backupfile = os.path.join(config['dirs']['backups'], backupname + "." + str(i))
             if not os.path.exists(backupfile):
-                os.rename(src = fname, dst = backupfile)
+                os.rename(fname, backupfile)
                 break
             elif i >= 100:
                 print("Somehow it's really hard to find a name for the backup file...")
@@ -65,6 +65,8 @@ def acme(name, domains):
     make_backup(certfile(name))
     with open(certfile(name), 'wb') as f:
         f.write(signed_crt)
+    # clean up
+    os.remove(csrfile(name))
 
 def request_cert(name):
     global config
@@ -100,9 +102,9 @@ def check_staging():
     print("Unstaging '{}' to '{}'".format(staging, live))
     # unstage the key!
     make_backup(keyfile(live))
-    os.rename(src = keyfile(staging), dst = keyfile(live))
+    os.rename(keyfile(staging), keyfile(live))
     make_backup(certfile(live))
-    os.rename(src = certfile(staging), dst = certfile(live))
+    os.rename(certfile(staging), certfile(live))
     return 2
 
 def auto_renewal():