From 4b0f91c1677803027d5e6085114ef7ebe0cf4a4f Mon Sep 17 00:00:00 2001 From: Maniack Crudelis Date: Wed, 14 Dec 2016 16:10:45 +0100 Subject: [PATCH] Usage des helpers --- scripts/.fonctions | 27 +++++++++++++++------------ scripts/backup | 7 +++++-- scripts/install | 4 ++-- scripts/remove | 2 +- scripts/restore | 7 +++++-- scripts/upgrade | 7 +++++-- 6 files changed, 33 insertions(+), 21 deletions(-) diff --git a/scripts/.fonctions b/scripts/.fonctions index fcd5b76..5475459 100755 --- a/scripts/.fonctions +++ b/scripts/.fonctions @@ -16,7 +16,7 @@ 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 - exit 1 + ynh_die } TRAP_ON () { # Activate signal capture @@ -32,7 +32,7 @@ TRAP_OFF () { # Ignoring signal capture until TRAP_ON CHECK_USER () { # Vérifie la validité de l'user admin # $1 = Variable de l'user admin. - sudo yunohost user list --json | grep -q "\"username\": \"$1\"" || (echo "Wrong admin" >&2 && false) + ynh_user_exists "$1" || (echo "Wrong admin" >&2 && false) } CHECK_PATH () { # Vérifie la présence du / en début de path. Et son absence à la fin. @@ -59,14 +59,15 @@ CHECK_FINALPATH () { # Vérifie que le dossier de destination n'est pas déjà u GENERATE_DB () { # Créer une base de données et un utilisateur dédié au nom de l'app. # $1 = Nom de la base de donnée - # Génère un mot de passe aléatoire. db_user=$1 - db_pwd=$(head -n20 /dev/urandom | tr -c -d 'A-Za-z0-9' | head -c20) + # Génère un mot de passe aléatoire. +# db_pwd=$(head -n20 /dev/urandom | tr -c -d 'A-Za-z0-9' | head -c20) + db_pwd=$(ynh_string_random) CHECK_VAR "$db_pwd" "db_pwd empty" # Utilise '$app' comme nom d'utilisateur et de base de donnée # Initialise la base de donnée et stocke le mot de passe mysql. ynh_mysql_create_db $db_user $db_user $db_pwd - sudo yunohost app setting $app mysqlpwd -v $db_pwd + ynh_app_setting_set $app mysqlpwd $db_pwd } SETUP_SOURCE () { # Télécharge la source, décompresse et copie dans $final_path @@ -91,7 +92,7 @@ SETUP_SOURCE () { # Télécharge la source, décompresse et copie dans $final_pa } 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 + if ! ynh_system_user_exists "$app" # 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 @@ -113,13 +114,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 $(sudo md5sum "$2" | cut -d' ' -f1) + ynh_app_setting_set $app $1_file_md5 $(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)" != $(sudo md5sum "$2" | cut -d' ' -f1) ]; then + if [ "$(ynh_app_setting_get $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 } @@ -164,7 +165,7 @@ REMOVE_LOGROTATE_CONF () { # Suppression de la configuration de logrotate } SECURE_REMOVE () { # Suppression de dossier avec vérification des variables - chaine="$1" + 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 @@ -180,8 +181,10 @@ SECURE_REMOVE () { # Suppression de dossier avec vérification des variable done if [ "$no_var" -eq 1 ] then - echo "Delete directory $chaine" - sudo rm -r "$chaine" + if [ -e "$chaine" ]; then + echo "Delete directory $chaine" + sudo rm -r "$chaine" + fi return 0 else echo "No detected variable." >&2 @@ -201,7 +204,7 @@ REMOVE_BDD () { # Suppression de la base de donnée et de l'utilisateur associé } REMOVE_SYS_USER () { # Supprime l'utilisateur système dédié à l'app - if grep -q "^$app:" /etc/passwd # Test l'existence de l'utilisateur + if ynh_system_user_exists "$app" # Test l'existence de l'utilisateur then sudo userdel $app fi diff --git a/scripts/backup b/scripts/backup index b4e5f4f..11d8a0b 100644 --- a/scripts/backup +++ b/scripts/backup @@ -3,8 +3,11 @@ # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -version=$(sudo yunohost app setting $app version) -port=$(sudo yunohost app setting $app port) +# Source app helpers +source /usr/share/yunohost/helpers + +version=$(ynh_app_setting_get $app version) +port=$(ynh_app_setting_get $app port) # The parameter $1 is the backup directory location diff --git a/scripts/install b/scripts/install index fb0d4b5..1d95a77 100644 --- a/scripts/install +++ b/scripts/install @@ -28,8 +28,8 @@ sudo yunohost firewall allow --no-upnp TCP $port > /dev/null 2>&1 sudo yunohost firewall allow --no-upnp UDP 1900 > /dev/null 2>&1 # Découverte SSDP pour UPNP. # Enregistre les infos dans la config YunoHost -sudo yunohost app setting $app version -v ${version:0:1} -sudo yunohost app setting $app port -v $port +ynh_app_setting_set $app version ${version:0:1} +ynh_app_setting_set $app port $port # Création du dossier yunohost.multimedia wget -nv https://github.com/maniackcrudelis/yunohost.multimedia/archive/master.zip diff --git a/scripts/remove b/scripts/remove index 145c41d..ab8f68a 100755 --- a/scripts/remove +++ b/scripts/remove @@ -8,7 +8,7 @@ app=$YNH_APP_INSTANCE_NAME # Source app helpers source /usr/share/yunohost/helpers -port=$(sudo yunohost app setting $app port) +port=$(ynh_app_setting_get $app port) # Ferme les ports dans le firewall if sudo yunohost firewall list | grep -q "\- $port$" diff --git a/scripts/restore b/scripts/restore index 2f8e80b..334f266 100644 --- a/scripts/restore +++ b/scripts/restore @@ -3,8 +3,11 @@ # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -version=$(sudo yunohost app setting $app version) -port=$(sudo yunohost app setting $app port) +# Source app helpers +source /usr/share/yunohost/helpers + +version=$(ynh_app_setting_get $app version) +port=$(ynh_app_setting_get $app port) # The parameter $1 is the uncompressed restore directory location backup_dir=$1/apps/$app diff --git a/scripts/upgrade b/scripts/upgrade index 8e1ddf0..ace41bc 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -5,8 +5,11 @@ source .fonctions # Charge les fonctions génériques habituellement utilisées # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -version=$(sudo yunohost app setting $app version) -port=$(sudo yunohost app setting $app port) +# Source app helpers +source /usr/share/yunohost/helpers + +version=$(ynh_app_setting_get $app version) +port=$(ynh_app_setting_get $app port) # Mise à jour du dossier yunohost.multimedia wget -nv https://github.com/maniackcrudelis/yunohost.multimedia/archive/master.zip