From 0d2ed6aa87ff473a688cb7171a357cc92503d9b8 Mon Sep 17 00:00:00 2001
From: Shuai Lin <linshuai2012@gmail.com>
Date: Fri, 25 Nov 2016 15:45:47 +0800
Subject: [PATCH] Only re-run letsencrypt verfication when necessary.

---
 scripts/bootstrap.py      | 34 ++++++++++++++++++++++------------
 scripts/utils/__init__.py |  7 +++++++
 2 files changed, 29 insertions(+), 12 deletions(-)

diff --git a/scripts/bootstrap.py b/scripts/bootstrap.py
index 87439df..7890ccf 100755
--- a/scripts/bootstrap.py
+++ b/scripts/bootstrap.py
@@ -17,7 +17,8 @@ import time
 
 from utils import (
     call, get_conf, get_install_dir, show_progress,
-    get_script, render_template, get_seafile_version, eprint
+    get_script, render_template, get_seafile_version, eprint,
+    cert_has_valid_days
 )
 
 seafile_version = get_seafile_version()
@@ -33,12 +34,30 @@ def init_letsencrypt():
         os.mkdir(ssl_dir)
 
     domain = get_conf('server.hostname')
+
+    context = {
+        'ssl_dir': ssl_dir,
+        'domain': domain,
+    }
+    render_template(
+        '/templates/letsencrypt.cron.template',
+        join(generated_dir, 'letsencrypt.cron'),
+        context
+    )
+
+    ssl_crt = '/shared/ssl/{}.crt'.format(domain)
+    if exists(ssl_crt):
+        show_progress('Found existing cert file {}'.format(ssl_crt))
+        if cert_has_valid_days(ssl_crt, 30):
+            show_progress('Skip letsencrypt verification since we have a valid certificate')
+            return
+
+    show_progress('Starting letsencrypt verification')
+    # Create a temporary nginx conf to start a server, which would accessed by letsencrypt
     context = {
         'https': False,
         'domain': domain,
     }
-
-    # Create a temporary nginx conf to start a server, which would accessed by letsencrypt
     render_template('/templates/seafile.nginx.conf.template',
                     '/etc/nginx/sites-enabled/seafile.nginx.conf', context)
 
@@ -53,15 +72,6 @@ def init_letsencrypt():
     #     time.sleep(1000)
     #     sys.exit(1)
 
-    context = {
-        'ssl_dir': ssl_dir,
-        'domain': domain,
-    }
-    render_template(
-        '/templates/letsencrypt.cron.template',
-        join(generated_dir, 'letsencrypt.cron'),
-        context
-    )
 
 def generate_local_nginx_conf():
     # Now create the final nginx configuratin
diff --git a/scripts/utils/__init__.py b/scripts/utils/__init__.py
index 3ebcd43..6866318 100644
--- a/scripts/utils/__init__.py
+++ b/scripts/utils/__init__.py
@@ -240,3 +240,10 @@ def render_template(template, target, context):
 def show_progress(msg):
     msg = '[{}] {}'.format(datetime.datetime.now().strftime('%Y-%m-%d %H:%M:%S'), green(msg))
     eprint(msg)
+
+def cert_has_valid_days(cert, days):
+    assert exists(cert)
+
+    secs = 86400 * int(days)
+    retcode = call('openssl x509 -checkend {} -noout -in {}'.format(secs, cert), check_call=False)
+    return retcode == 0