From 00cad842f28004d57d20d12198b3e528c568a0f8 Mon Sep 17 00:00:00 2001
From: magikcypress <cyp@rouquin.me>
Date: Tue, 21 Feb 2017 04:52:10 +0100
Subject: [PATCH] Mieux traiter la suppression de l'app

---
 scripts/.fonctions | 30 ++++++++++++++++++++++++------
 scripts/remove     |  1 +
 2 files changed, 25 insertions(+), 6 deletions(-)

diff --git a/scripts/.fonctions b/scripts/.fonctions
index 76df5ac..22a8e0b 100644
--- a/scripts/.fonctions
+++ b/scripts/.fonctions
@@ -165,13 +165,31 @@ REMOVE_LOGROTATE_CONF () {	# Suppression de la configuration de logrotate
 	fi
 }
 
-REMOVE_APP_DIR () {	# Suppression du dossier de l'application
-	if test -n "$app"
-	then	# Supprime le dossier seulement si $app n'est pas vide.
-		if [ -e "/var/www/$app" ]; then	# Delete final_path
-			echo "Delete app dir"
-			sudo rm -r "/var/www/$app"
+SECURE_REMOVE () {      # Suppression de dossier avec vérification des variables
+	chaine="$1"	# L'argument doit être donné entre quotes simple '', pour éviter d'interpréter les variables.
+	no_var=0
+	while (echo "$chaine" | grep -q '\$')	# Boucle tant qu'il y a des $ dans la chaine
+	do
+		no_var=1
+		global_var=$(echo "$chaine" | cut -d '$' -f 2)	# Isole la première variable trouvée.
+		only_var=\$$(expr "$global_var" : '\([A-Za-z0-9_]*\)')	# Isole complètement la variable en ajoutant le $ au début et en gardant uniquement le nom de la variable. Se débarrasse surtout du / et d'un éventuel chemin derrière.
+		real_var=$(eval "echo ${only_var}")		# `eval "echo ${var}` permet d'interpréter une variable contenue dans une variable.
+		if test -z "$real_var" || [ "$real_var" = "/" ]; then
+			echo "Variable $only_var is empty, suppression of $chaine cancelled." >&2
+			return 1
 		fi
+		chaine=$(echo "$chaine" | sed "s@$only_var@$real_var@")	# remplace la variable par sa valeur dans la chaine.
+	done
+	if [ "$no_var" -eq 1 ]
+	then
+		if [ -e "$chaine" ]; then
+			echo "Delete directory $chaine"
+			sudo rm -r "$chaine"
+		fi
+		return 0
+	else
+		echo "No detected variable." >&2
+		return 1
 	fi
 }
 
diff --git a/scripts/remove b/scripts/remove
index 9e18667..3ed2b1b 100644
--- a/scripts/remove
+++ b/scripts/remove
@@ -15,6 +15,7 @@ domain=$(ynh_app_setting_set $app domain)
 REMOVE_BDD $app	# Suppression de la base de donnée et de l'utilisateur associé.
 
 REMOVE_APP_DIR	# Suppression du dossier de l'application
+SECURE_REMOVE '/var/www/$app'	# Suppression du dossier de l'application
 
 REMOVE_NGINX_CONF	# Suppression de la configuration nginx