From 815c06a33bc45a81c1a22d0f3c27d1f996638a53 Mon Sep 17 00:00:00 2001 From: Maniack Crudelis Date: Wed, 14 Dec 2016 16:10:00 +0100 Subject: [PATCH] Usage des helpers et fix restore --- manifest2.2.json | 67 ---------------------------------------------- scripts/.fonctions | 27 ++++++++++--------- scripts/backup | 9 ++++--- scripts/install | 18 ++++++------- scripts/remove | 2 +- scripts/restore | 14 +++++++--- scripts/upgrade | 22 ++++++++------- 7 files changed, 54 insertions(+), 105 deletions(-) delete mode 100644 manifest2.2.json diff --git a/manifest2.2.json b/manifest2.2.json deleted file mode 100644 index 755e2b8..0000000 --- a/manifest2.2.json +++ /dev/null @@ -1,67 +0,0 @@ -{ - "name": "Lutim", - "id": "lutim", - "packaging_format": 1, - "description": { - "en": "Self hosting images and sharing anonymous application", - "fr": "Application d'hébergement et de partage d'images anonyme" - }, - "version": "0.6 dev", - "url": "https://lut.im", - "licence": "free", - "maintainer": { - "name": "Maniack Crudelis et matlink", - "email": "maniackc_dev@crudelis.fr matlink@matlink.fr" - }, - "multi_instance": "false", - "services": [ - "nginx" - ], - "arguments": { - "install" : [ - { - "name": "domain", - "ask": { - "en": "Choose a domain for Lutim", - "fr": "Choisissez un domaine pour Lutim" - }, - "example": "domain.org" - }, - { - "name": "path", - "ask": { - "en": "Choose a path for Lutim", - "fr": "Choisissez un chemin pour Lutim" - }, - "example": "/lutim", - "default": "/lutim" - }, - { - "name": "admin", - "ask": { - "en": "Choose the Lutim administrator (must be an existing YunoHost user)", - "fr": "Choisissez un administrateur Lutim (doit être un utilisateur YunoHost)" - }, - "example": "john" - }, - { - "name": "is_public", - "ask": { - "en": "Uploading images is it public?", - "fr": "L'upload des images est-il public ?" - }, - "choices": ["Yes", "No"], - "default": "No" - }, - { - "name": "always_encrypt", - "ask": { - "en": "Force the encryption of images?", - "fr": "Forcer le chiffrement des images ?" - }, - "choices": ["Yes", "No"], - "default": "Yes" - } - ] - } -} 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 2988cfa..dfc0d88 100644 --- a/scripts/backup +++ b/scripts/backup @@ -3,9 +3,12 @@ # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -final_path=$(sudo yunohost app setting $app final_path) -domain=$(sudo yunohost app setting $app domain) -codename=$(sudo yunohost app setting $app codename) +# Source app helpers +source /usr/share/yunohost/helpers + +final_path=$(ynh_app_setting_get $app final_path) +domain=$(ynh_app_setting_get $app domain) +codename=$(ynh_app_setting_get $app codename) # The parameter $1 is the backup directory location # which will be compressed afterward diff --git a/scripts/install b/scripts/install index 302b0d8..36a72c3 100644 --- a/scripts/install +++ b/scripts/install @@ -42,16 +42,16 @@ CHECK_VAR "$domain_regex" "domain_regex empty" FIND_PORT 8095 # Cherche un port libre. # Enregistre les infos dans la config YunoHost -sudo yunohost app setting $app admin -v $admin -sudo yunohost app setting $app domain -v $domain -sudo yunohost app setting $app is_public -v $is_public -sudo yunohost app setting $app port -v $port -sudo yunohost app setting $app always_encrypt -v $always_encrypt +ynh_app_setting_set $app admin $admin +ynh_app_setting_set $app domain $domain +ynh_app_setting_set $app is_public $is_public +ynh_app_setting_set $app port $port +ynh_app_setting_set $app always_encrypt $always_encrypt # Créer le repertoire de destination et stocke son emplacement. sudo mkdir "$final_path" -sudo yunohost app setting $app final_path -v $final_path +ynh_app_setting_set $app final_path $final_path SETUP_SOURCE "lutim.tar.gz" # Télécharge la source, décompresse et copie dans $final_path @@ -82,7 +82,7 @@ STORE_MD5_CONFIG "lutim.conf" "$final_path/lutim.conf" # Enregistre la somme de codename=$(lsb_release -a 2>/dev/null | grep Codename | cut -f 2) CHECK_VAR "$codename" "codename empty" -sudo yunohost app setting $app codename -v $codename +ynh_app_setting_set $app codename $codename if [ "$codename" = "wheezy" ] then # On utilise le script init pour wheezy. # Mise en place du script init @@ -131,13 +131,13 @@ sudo sed -i "s@__PATH__@$path@g" /etc/nginx/conf.d/$domain.d/$app.conf sudo sed -i "s@__PORT__@$port@g" /etc/nginx/conf.d/$domain.d/$app.conf # Make app public or private -sudo yunohost app setting $app skipped_uris -v "/" +ynh_app_setting_set $app skipped_uris "/" if [ "$is_public" = "No" ]; then # Si l'app est privée, seul le visionnage des images reste public if [ "$path" == "/" ]; then path="" # Si path correspond à la racine, supprime le / pour éviter une erreur de la regex. fi - sudo yunohost app setting $app protected_regex -v "$domain_regex$path/stats$","$domain_regex$path/manifest.webapp$","$domain_regex$path/$","$domain_regex$path/d/.*$","$domain_regex$path/m/.*$" + ynh_app_setting_set $app protected_regex "$domain_regex$path/stats$","$domain_regex$path/manifest.webapp$","$domain_regex$path/$","$domain_regex$path/d/.*$","$domain_regex$path/m/.*$" fi # Making log symbolic link to /var/log diff --git a/scripts/remove b/scripts/remove index 6bf4711..3377a9e 100644 --- a/scripts/remove +++ b/scripts/remove @@ -8,7 +8,7 @@ app=$YNH_APP_INSTANCE_NAME # Source app helpers source /usr/share/yunohost/helpers -domain=$(sudo yunohost app setting $app domain) +domain=$(ynh_app_setting_get $app domain) # Arrêt du service if [ -e "/etc/init.d/lutim" ]; then diff --git a/scripts/restore b/scripts/restore index f5bfdf8..c04609a 100644 --- a/scripts/restore +++ b/scripts/restore @@ -3,9 +3,12 @@ # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -final_path=$(sudo yunohost app setting $app final_path) -domain=$(sudo yunohost app setting $app domain) -codename=$(sudo yunohost app setting $app codename) +# Source app helpers +source /usr/share/yunohost/helpers + +final_path=$(ynh_app_setting_get $app final_path) +domain=$(ynh_app_setting_get $app domain) +codename=$(ynh_app_setting_get $app codename) # The parameter $1 is the uncompressed restore directory location backup_dir=$1/apps/$app @@ -18,6 +21,11 @@ sudo cp -a $backup_dir/yunohost/. /etc/yunohost/apps/$app sudo cp -a $backup_dir/nginx.conf /etc/nginx/conf.d/$domain.d/$app.conf +# Installation de perlmagick, interface perl pour imagemagick et de carton, gestionnaire de dépendances perl +sudo apt-get update +sudo apt-get install carton perlmagick -qy + + if [ "$codename" = "wheezy" ] then # Restauration des fichiers du script init diff --git a/scripts/upgrade b/scripts/upgrade index 820beb2..866d55d 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -5,12 +5,15 @@ source .fonctions # Charge les fonctions génériques habituellement utilisées # Récupère les infos de l'application. app=$YNH_APP_INSTANCE_NAME -domain=$(sudo yunohost app setting $app domain) -path=$(sudo yunohost app setting $app path) -is_public=$(sudo yunohost app setting $app is_public) -port=$(sudo yunohost app setting $app port) -always_encrypt=$(sudo yunohost app setting $app always_encrypt) -final_path=$(sudo yunohost app setting $app final_path) +# Source app helpers +source /usr/share/yunohost/helpers + +domain=$(ynh_app_setting_get $app domain) +path=$(ynh_app_setting_get $app path) +is_public=$(ynh_app_setting_get $app is_public) +port=$(ynh_app_setting_get $app port) +always_encrypt=$(ynh_app_setting_get $app always_encrypt) +final_path=$(ynh_app_setting_get $app final_path) if [ "${#final_path}" -eq 0 ] then # Si final_path n'est pas renseigné dans la config yunohost, cas d'ancien script, code final_path en dur @@ -46,7 +49,7 @@ sudo sed -i "s@__ENCRYPT__@$always_encrypt@g" "$final_path/lutim.conf" STORE_MD5_CONFIG "lutim.conf" "$final_path/lutim.conf" # Réenregistre la somme de contrôle du fichier de config codename=$(lsb_release -a 2>/dev/null | grep Codename | cut -f 2) -sudo yunohost app setting $app codename -v $codename +ynh_app_setting_set $app codename $codename if [ "$codename" = "wheezy" ] then # On utilise le script init pour wheezy. # Mise en place du script init @@ -70,7 +73,6 @@ else # Et le script systemd à partir de jessie sudo sed -i "s@__FINALPATH__@$final_path/@g" /etc/systemd/system/lutim.service ## Démarrage auto du service sudo systemctl enable lutim.service - fi ## Mise en place des crons @@ -86,10 +88,10 @@ cd $final_path sudo carton install 2>&1 | sudo tee -a "/var/log/$app/setup_carton.log" # Make app public or private -sudo yunohost app setting $app skipped_uris -v "/" +ynh_app_setting_set $app skipped_uris "/" if [ "$is_public" = "No" ]; then # Si l'app est privée, seul le visionnage des images reste public - sudo yunohost app setting $app protected_regex -v "$domain_regex$path/stats$","$domain_regex$path/manifest.webapp$","$domain_regex$path/$","$domain_regex$path/d/.*$","$domain_regex$path/m/.*$" + ynh_app_setting_set $app protected_regex "$domain_regex$path/stats$","$domain_regex$path/manifest.webapp$","$domain_regex$path/$","$domain_regex$path/d/.*$","$domain_regex$path/m/.*$" fi # Configure les droits d'accès au fichiers