From 37f3bff0d9cc2b0c9726842ba6b65a7d05317a4c Mon Sep 17 00:00:00 2001
From: Jephte CLAIN <Jephte.Clain@univ-reunion.fr>
Date: Mon, 2 Nov 2015 10:59:48 +0400
Subject: [PATCH] apacheconfig: diverses corrections de bugs

---
 apacheconfig                                   | 18 +++++++++++-------
 .../apacheconfig/certsconf/default-certs.conf  |  9 +++++----
 2 files changed, 16 insertions(+), 11 deletions(-)

diff --git a/apacheconfig b/apacheconfig
index d6b0862..efe1a97 100755
--- a/apacheconfig
+++ b/apacheconfig
@@ -47,17 +47,19 @@ OPTIONS
         Les arguments qui restent sont passés tels quels à apache_autoconf
     -r, --certsdir CERTSDIR
         Spécifier le cas échéant le répertoire contenant les certificats à
-        déployer."
+        déployer. Cet argument est requis si le répertoire certsconf/ existe."
 }
 
 # toujours placer une variable dépendante AVANT la variable maitre
 TEMPLATE_STATIC_VARS=(
     hostname aliases host
+    certsdir caname
 )
 TEMPLATE_DYNAMIC_VARS=(
     ips_namevirtualhosts ips_listens ips
     dhostname daliases dhost
-    admin certsdir caname certname keyname configdir)
+    admin configdir
+)
 TEMPLATE_NOWRITE_VARS=(hostname aliases dhostname daliases configdir)
 
 __TEMPLATE_DEFAULTF_host=__template_defaultf_host
@@ -127,7 +129,8 @@ TEMPLATECTL_SRCDIRS=(apacheconfig)
 TEMPLATECTL_CONFIG="$TEMPLATECTL_NAME"
 TEMPLATECTL_DEFAULTS=(
     admin=supervision-gdrsi@listes.univ-reunion.fr
-    certsdir=renater
+    certsdir=1507-renater
+    caname=1507-DigiCertCA.crt
 )
 TEMPLATECTL_VARS=()
 
@@ -135,7 +138,7 @@ action=
 destdir=
 nohideconfig=
 templateopt=
-certsdir=
+aac_certsdir=
 args=(
     --help '$exit_with display_help'
     -c,--create action=create
@@ -156,7 +159,7 @@ args=(
     -7,--wheezy '$array_add TEMPLATECTL_VARS sysver=wheezy'
     -8,--jessie '$array_add TEMPLATECTL_VARS sysver=jessie'
     -u,--update,--deploy action=deploy
-    -r:,--certsdir: certsdir=
+    -r:,--certsdir: aac_certsdir=
 )
 parse_args "$@"; set -- "${args[@]}"
 
@@ -192,6 +195,7 @@ if [ "$action" == create ]; then
     fi
     [ -d "$destdir" ] || die "$destdir: répertoire introuvable"
 
+    [ -n "$host" ] || host="$1"
     if [ -n "$host" ]; then
         read_value -i "Veuillez entrer le nom d'hôte" host "$host"
     else
@@ -217,7 +221,7 @@ $__TEMPLATECTL_HELP"
 elif [ "$action" == deploy ]; then
     [ -d "$destdir" ] || die "$destdir: répertoire introuvable"
 
-    args=(-d "$destdir" --deploy ${certsdir:+-r "$certsdir"})
+    args=(-d "$destdir" --deploy ${aac_certsdir:+-r "$aac_certsdir"})
     for __name in "${TEMPLATE_DYNAMIC_VARS[@]}"; do
         array_add args -v "$__name=${!__name}"
     done
@@ -225,7 +229,7 @@ elif [ "$action" == deploy ]; then
     run_as_root "${args[@]}"
 
     etitle "Mise à jour du système"
-    args=("$destdir" "$certsdir")
+    args=("$destdir" "$aac_certsdir")
     for __name in "${TEMPLATE_DYNAMIC_VARS[@]}"; do
         array_add args "$__name=${!__name}"
     done
diff --git a/lib/ulib/templates/apacheconfig/certsconf/default-certs.conf b/lib/ulib/templates/apacheconfig/certsconf/default-certs.conf
index 1074ed9..2ca6ace 100644
--- a/lib/ulib/templates/apacheconfig/certsconf/default-certs.conf
+++ b/lib/ulib/templates/apacheconfig/certsconf/default-certs.conf
@@ -1,7 +1,8 @@
 # -*- coding: utf-8 mode: sh -*- vim:sw=4:sts=4:et:ai:si:sta:fenc=utf-8
 
-# Ce fichier permet de configurer quels certificats du répertoire $certsdir il
-# faut installer sur le serveur.
+# Cette variable est utilisée par la fonction refcerts() du script runs. C'est
+# le nom d'un répertoire à chercher dans RUNSMODULESPATH qui contient les
+# certificats à installer sur le serveur.
 certsdir=@@certsdir@@
 
 # Fichier contenant les certificats racines qui valident le certificat à
@@ -10,5 +11,5 @@ certsdir=@@certsdir@@
 ca=@@caname@@
 
 # Certificat et clé privée à installer
-cert=@@certname@@
-key=@@keyname@@
+cert=
+key=