summaryrefslogtreecommitdiff
path: root/roles/certificate/tasks
diff options
context:
space:
mode:
Diffstat (limited to 'roles/certificate/tasks')
-rw-r--r--roles/certificate/tasks/main.yml11
1 files changed, 11 insertions, 0 deletions
diff --git a/roles/certificate/tasks/main.yml b/roles/certificate/tasks/main.yml
index 5d25a63..a736e6b 100644
--- a/roles/certificate/tasks/main.yml
+++ b/roles/certificate/tasks/main.yml
@@ -13,3 +13,14 @@
-d {{ domains | join(' -d ') }}
args:
creates: '/etc/letsencrypt/live/{{ domains | first }}/fullchain.pem'
+
+# TODO: rewrite this role or make it more generic.
+#
+# Currently we reuse certificate role for web and mail servers, but they
+# are actually quite different.
+# * We can't start nginx without valid certificate, but can start postfix (and
+# dovecot?).
+# * We can serve acme challenge with nginx itself on renewal, but we need
+# `--standalone` mode for mail server (or need to bring nginx).
+# * It is undesirable to bring down nginx any time we are updating
+# SSL certificates.