diff --git a/README.md b/README.md index e92a4b9..21a1865 100644 --- a/README.md +++ b/README.md @@ -12,6 +12,6 @@ Le script installe les paquets *i965-va-driver*, *i965-va-driver*, *libavcodec56 [Documentation minidlna](https://yunohost.org/#/app_minidlna_fr) pour Yunohost. **Mise à jour du package:** -sudo yunohost app upgrade -u https://github.com/YunoHost-Apps/minidlna_ynh +sudo yunohost app upgrade --verbose minidlna -u https://github.com/YunoHost-Apps/minidlna_ynh **Multi-utilisateur:** Pas de gestion des utilisateurs, les partages sont accessible librement. diff --git a/scripts/.fonctions b/scripts/.fonctions index 2614347..f93eb3c 100755 --- a/scripts/.fonctions +++ b/scripts/.fonctions @@ -1,9 +1,5 @@ #!/bin/bash -YNH_VERSION () { # Renvoi le numéro de version de la moulinette Yunohost - ynh_version=$(sudo yunohost -v | grep "moulinette:" | cut -d' ' -f2 | cut -d'.' -f1,2) -} - CHECK_VAR () { # Vérifie que la variable n'est pas vide. # $1 = Variable à vérifier # $2 = Texte à afficher en cas d'erreur @@ -20,10 +16,6 @@ EXIT_PROPERLY () { # Provoque l'arrêt du script en cas d'erreur. Et nettoye les # Compense le bug de ssowat qui ne supprime pas l'entrée de l'app en cas d'erreur d'installation. sudo sed -i "\@\"$domain$path/\":@d" /etc/ssowat/conf.json - if [ $ynh_version = "2.2" ]; then - /bin/bash $script_dir/remove # Appel le script remove. En 2.2, ce comportement n'est pas automatique. - fi - exit 1 } @@ -93,12 +85,22 @@ SETUP_SOURCE () { # Télécharge la source, décompresse et copie dans $final_pa # Copie les fichiers sources sudo cp -a $(cat ../sources/source_dir)/. "$final_path" # Copie les fichiers additionnels ou modifiés. - sudo cp -a ../sources/ajouts/. "$final_path" + if test -e "../sources/ajouts"; then + sudo cp -a ../sources/ajouts/. "$final_path" + fi +} + +ADD_SYS_USER () { # Créer un utilisateur système dédié à l'app + if grep -q "^$app:" /etc/passwd # Test l'existence de l'utilisateur + then + sudo useradd -d /var/www/$app --system --user-group $app --shell /usr/sbin/nologin || (echo "Unable to create $app system account" >&2 && false) + fi } POOL_FPM () { # Créer le fichier de configuration du pool php-fpm et le configure. sed -i "s@__NAMETOCHANGE__@$app@g" ../conf/php-fpm.conf sed -i "s@__FINALPATH__@$final_path@g" ../conf/php-fpm.conf + sed -i "s@__USER__@$app@g" ../conf/php-fpm.conf finalphpconf=/etc/php5/fpm/pool.d/$app.conf sudo cp ../conf/php-fpm.conf $finalphpconf sudo chown root: $finalphpconf @@ -111,13 +113,13 @@ POOL_FPM () { # Créer le fichier de configuration du pool php-fpm et le configu STORE_MD5_CONFIG () { # Enregistre la somme de contrôle du fichier de config # $1 = Nom du fichier de conf pour le stockage dans settings.yml # $2 = Nom complet et chemin du fichier de conf. - sudo yunohost app setting $app $1_file_md5 -v $(md5sum "$2" | cut -d' ' -f1) + sudo yunohost app setting $app $1_file_md5 -v $(sudo md5sum "$2" | cut -d' ' -f1) } CHECK_MD5_CONFIG () { # Créé un backup du fichier de config si il a été modifié. # $1 = Nom du fichier de conf pour le stockage dans settings.yml # $2 = Nom complet et chemin du fichier de conf. - if [ "$(sudo yunohost app setting $app $1_file_md5)" != $(md5sum "$2" | cut -d' ' -f1) ]; then + if [ "$(sudo yunohost app setting $app $1_file_md5)" != $(sudo md5sum "$2" | cut -d' ' -f1) ]; then sudo cp -a "$2" "$2.backup.$(date '+%d.%m.%y_%Hh%M,%Ss')" # Si le fichier de config a été modifié, créer un backup. fi } @@ -161,13 +163,29 @@ 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" + 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 + echo "Delete directory $chaine" + sudo rm -r "$chaine" + return 0 + else + echo "No detected variable." >&2 + return 1 fi } @@ -181,3 +199,10 @@ REMOVE_BDD () { # Suppression de la base de donnée et de l'utilisateur associé ynh_mysql_drop_user $db_user fi } + +REMOVE_SYS_USER () { # Supprime l'utilisateur système dédié à l'app + if grep -q "^$app:" /etc/passwd # Test l'existence de l'utilisateur + then + sudo userdel $app + fi +}