of course it is okay for that dir to already exist...

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

diff --git a/letsencrypt-tiny b/letsencrypt-tiny
index 7f23ad5cc0e0242c10272dad17533792a073c529..eef52e7548cc113223c005e073ff2e9b2bbfbdeb 100755 (executable)
--- a/letsencrypt-tiny
+++ b/letsencrypt-tiny
@@ -1,5 +1,5 @@
 #!/usr/bin/env python3
 #!/usr/bin/env python3

-## Call with "--help" for documentation.
+## See <https://www.ralfj.de/blog/2017/12/26/lets-encrypt.html> for documentation.

 
 import argparse, configparser, itertools, stat, os, os.path, sys, subprocess, datetime
 
 
 import argparse, configparser, itertools, stat, os, os.path, sys, subprocess, datetime
 


@@ -21,6 +21,7 @@ def keyfile(name):
 
 def make_backup(fname):
     if os.path.exists(fname):
 
 def make_backup(fname):
     if os.path.exists(fname):

+        os.makedirs(config['dirs']['backups'], exist_ok = True)

         backupname = os.path.basename(fname) + "." + str(datetime.date.today())
         i = 0
         while True:
         backupname = os.path.basename(fname) + "." + str(datetime.date.today())
         i = 0
         while True:


@@ -64,7 +65,7 @@ def acme(keyfilename, certfilename, domains):
     try:
         # call acme-tiny as a script
         acme_tiny = os.path.join(os.path.dirname(os.path.realpath(__file__)), 'acme-tiny', 'acme_tiny.py')
     try:
         # call acme-tiny as a script
         acme_tiny = os.path.join(os.path.dirname(os.path.realpath(__file__)), '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']])
+        signed_crt = subprocess.check_output(["python3", 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:
         # save new certificate
         make_backup(certfilename)
         with open(certfilename, 'wb') as f:


@@ -123,9 +124,12 @@ def auto_renewal(live, staging):
     # determine what to do
     now = datetime.datetime.now()
     key_age = now - key_mtime(live)
     # determine what to do
     now = datetime.datetime.now()
     key_age = now - key_mtime(live)

-    cert_validity = cert_expiry(live) - now

     need_new_key = key_age >= max_key_age
     need_new_key = key_age >= max_key_age

-    need_new_cert = cert_validity <= renew_cert_time
+    if os.path.exists(certfile(live)):
+        cert_validity = cert_expiry(live) - now
+        need_new_cert = cert_validity <= renew_cert_time
+    else:
+        need_new_cert = True

     if need_new_cert and key_age + renew_cert_time >= max_key_age:
         # We are about to request a new certificate, and within <renew_cert_time>, we need a new key: Get the new key now
         need_new_key = True
     if need_new_cert and key_age + renew_cert_time >= max_key_age:
         # We are about to request a new certificate, and within <renew_cert_time>, we need a new key: Get the new key now
         need_new_key = True