fix: make sure we don't add -d options for blank domains

ebarault · Mar 6, 2019 · 0a95d07 · 0a95d07
1 parent ad7179d
commit 0a95d07
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/scripts/run_certbot.sh b/scripts/run_certbot.sh
@@ -69,7 +69,11 @@ processCertificates() {
         # look for certificate additional domain names and append them as '-d <name>' (-d for certbot's --domains option)
         for altname in ${subjectaltnames}; do
           if [ "${altname}" != "${subject}" ]; then
-            domains="${domains} -d ${altname}"
+            if [ "${domains}" != "" ]; then
+              domains="${domains} -d ${altname}"
+            else
+              domains="${altname}"
+            fi   
           fi
         done