From 8c3bc7975eddbcca9bb714374092aa0a0a812421 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 4 Mar 2019 23:38:50 +0100
Subject: [PATCH 01/19] [fix] correct php config + migrate to php7

See https://github.com/YunoHost/example_ynh/pull/52
---
 conf/php-fpm.conf | 11 +++++++++++
 conf/php-fpm.ini  | 10 ----------
 scripts/backup    |  3 +--
 scripts/restore   |  3 +--
 4 files changed, 13 insertions(+), 14 deletions(-)
 delete mode 100644 conf/php-fpm.ini

diff --git a/conf/php-fpm.conf b/conf/php-fpm.conf
index 8affbc2..58ab5e2 100644
--- a/conf/php-fpm.conf
+++ b/conf/php-fpm.conf
@@ -390,3 +390,14 @@ catch_workers_output = yes
 ;php_admin_value[error_log] = /var/log/fpm-php.www.log
 ;php_admin_flag[log_errors] = on
 ;php_admin_value[memory_limit] = 32M
+
+; Common values to change to increase file upload limit
+; php_admin_value[upload_max_filesize] = 50M
+; php_admin_value[post_max_size] = 50M
+; php_admin_flag[mail.add_x_header] = Off
+
+; Other common parameters
+; php_admin_value[max_execution_time] = 600
+; php_admin_value[max_input_time] = 300
+; php_admin_value[memory_limit] = 256M
+; php_admin_flag[short_open_tag] = On
diff --git a/conf/php-fpm.ini b/conf/php-fpm.ini
deleted file mode 100644
index 5c27d59..0000000
--- a/conf/php-fpm.ini
+++ /dev/null
@@ -1,10 +0,0 @@
-; Common values to change to increase file upload limit
-; upload_max_filesize = 50M
-; post_max_size = 50M
-; mail.add_x_header = Off
-
-; Other common parameters
-; max_execution_time = 600
-; max_input_time = 300
-; memory_limit = 256M
-; short_open_tag = On
diff --git a/scripts/backup b/scripts/backup
index b28bec4..5fdc910 100644
--- a/scripts/backup
+++ b/scripts/backup
@@ -44,8 +44,7 @@ ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 # BACKUP THE PHP-FPM CONFIGURATION
 #=================================================
 
-ynh_backup "/etc/php5/fpm/pool.d/$app.conf"
-ynh_backup "/etc/php5/fpm/conf.d/20-$app.ini"
+ynh_backup "/etc/php7.0/fpm/pool.d/$app.conf"
 
 #=================================================
 # BACKUP THE MYSQL DATABASE
diff --git a/scripts/restore b/scripts/restore
index d205235..3ab1d08 100644
--- a/scripts/restore
+++ b/scripts/restore
@@ -78,8 +78,7 @@ chown -R $app: $final_path/var
 # RESTORE THE PHP-FPM CONFIGURATION
 #=================================================
 
-ynh_restore_file "/etc/php5/fpm/pool.d/$app.conf"
-ynh_restore_file "/etc/php5/fpm/conf.d/20-$app.ini"
+ynh_restore_file "/etc/php7.0/fpm/pool.d/$app.conf"
 
 #=================================================
 # RESTORE THE CRON FILE

From 4de12394071f14268be4508cdc76a1723751c58e Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 4 Mar 2019 23:51:16 +0100
Subject: [PATCH 02/19] [enh] Use helper 'ynh_print_info'

---
 scripts/backup  | 13 ++++++++++++-
 scripts/install | 28 ++++++++++++++++++++++++----
 scripts/remove  | 12 ++++++++++++
 scripts/restore | 27 +++++++++++++++++++--------
 scripts/upgrade | 19 +++++++++++++++++--
 5 files changed, 84 insertions(+), 15 deletions(-)

diff --git a/scripts/backup b/scripts/backup
index 5fdc910..30db841 100644
--- a/scripts/backup
+++ b/scripts/backup
@@ -19,6 +19,7 @@ ynh_abort_if_errors
 #=================================================
 # LOAD SETTINGS
 #=================================================
+ynh_print_info "Loading installation settings..."
 
 app=$YNH_APP_INSTANCE_NAME
 
@@ -31,24 +32,28 @@ db_name=$(ynh_app_setting_get $app db_name)
 #=================================================
 # BACKUP THE APP MAIN DIR
 #=================================================
+ynh_print_info "Backing up the main app directory..."
 
 ynh_backup "$final_path"
 
 #=================================================
 # BACKUP THE NGINX CONFIGURATION
 #=================================================
+ynh_print_info "Backing up nginx web server configuration..."
 
 ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 
 #=================================================
 # BACKUP THE PHP-FPM CONFIGURATION
 #=================================================
+ynh_print_info "Backing up php-fpm configuration..."
 
 ynh_backup "/etc/php7.0/fpm/pool.d/$app.conf"
 
 #=================================================
 # BACKUP THE MYSQL DATABASE
 #=================================================
+ynh_print_info "Backing up the MySQL database..."
 
 ynh_mysql_dump_db "$db_name" > db.sql
 
@@ -59,7 +64,13 @@ ynh_mysql_dump_db "$db_name" > db.sql
 #=================================================
 
 #=================================================
-# BACKUP THE CRON FILE
+# BACKUP A CRON FILE
 #=================================================
 
 ynh_backup "/etc/cron.d/$app"
+
+#=================================================
+# END OF SCRIPT
+#=================================================
+
+ynh_print_info "Backup script completed for $app. (YunoHost will then actually copy those files to the archive)."
diff --git a/scripts/install b/scripts/install
index a136bd9..b0e6329 100644
--- a/scripts/install
+++ b/scripts/install
@@ -40,6 +40,7 @@ app=$YNH_APP_INSTANCE_NAME
 #=================================================
 # CHECK IF THE APP CAN BE INSTALLED WITH THESE ARGS
 #=================================================
+ynh_print_info "Validating installation parameters..."
 
 final_path=/var/www/$app
 test ! -e "$final_path" || ynh_die "This path already contains a folder"
@@ -55,6 +56,7 @@ ynh_webpath_register $app $domain $path_url
 #=================================================
 # STORE SETTINGS FROM MANIFEST
 #=================================================
+ynh_print_info "Storing installation settings..."
 
 ynh_app_setting_set $app domain $domain
 ynh_app_setting_set $app path $path_url
@@ -65,8 +67,12 @@ ynh_app_setting_set $app is_public $is_public
 #=================================================
 # CREATE A MYSQL DATABASE
 #=================================================
-# If your app uses a MySQL database, you can use these lines to bootstrap
-# a database, an associated user and save the password in app settings
+ynh_print_info "Creating a MySQL database..."
+
+### Use these lines if you need a database for the application.
+### `ynh_mysql_setup_db` will create a database, an associated user and a ramdom password.
+### The password will be stored as 'mysqlpwd' into the app settings,
+### and will be available as $db_pwd
 
 db_name=$(ynh_sanitize_dbid $app)
 ynh_app_setting_set $app db_name $db_name
@@ -75,6 +81,7 @@ ynh_mysql_setup_db $db_name $db_name
 #=================================================
 # DOWNLOAD, CHECK AND UNPACK SOURCE
 #=================================================
+ynh_print_info "Setting up source files..."
 
 ynh_app_setting_set $app final_path $final_path
 # Download, check integrity, uncompress and patch the source from app.src
@@ -83,6 +90,7 @@ ynh_setup_source "$final_path"
 #=================================================
 # NGINX CONFIGURATION
 #=================================================
+ynh_print_info "Configuring nginx web server..."
 
 # Create a dedicated nginx config
 ynh_add_nginx_config
@@ -90,6 +98,7 @@ ynh_add_nginx_config
 #=================================================
 # CREATE DEDICATED USER
 #=================================================
+ynh_print_info "Configuring system user..."
 
 # Create a system user
 ynh_system_user_create $app
@@ -97,6 +106,7 @@ ynh_system_user_create $app
 #=================================================
 # PHP-FPM CONFIGURATION
 #=================================================
+ynh_print_info "Configuring php-fpm..."
 
 # Create a dedicated php-fpm config
 ynh_add_fpm_config
@@ -106,6 +116,7 @@ ynh_add_fpm_config
 #=================================================
 # Add cron job
 #=================================================
+ynh_print_info "Configuring a cron task..."
 
 # Path where crontab will be installed
 cron_path="/etc/cron.d/$app"
@@ -130,6 +141,7 @@ ynh_replace_string "__FINALPATH__" "$final_path" "$cron_path"
 chown -R $app: $final_path
 
 # Set the app as temporarily public for curl call
+ynh_print_info "Configuring SSOwat..."
 ynh_app_setting_set $app skipped_uris "/"
 # Reload SSOwat config
 yunohost app ssowatconf
@@ -137,10 +149,10 @@ yunohost app ssowatconf
 # Reload Nginx
 systemctl reload nginx
 
+# Installation with curl
+ynh_print_info "Finalizing installation..."
 # Get database password
 db_pwd=$(ynh_app_setting_get $app mysqlpwd)
-
-# Installation with curl
 ynh_local_curl "/index.php?mod=install" "password=$password" "confirmPassword=$password" "type=db" "db[host]=localhost" "db[user]=$db_name" "db[password]=$db_pwd" "db[dbname]=$db_name"
 
 # Remove the public access
@@ -165,6 +177,7 @@ chown -R $app: $final_path/static/media/annonce   # needed for "Mes annonces sau
 #=================================================
 # SETUP SSOWAT
 #=================================================
+ynh_print_info "Configuring SSOwat..."
 
 # Make app public if necessary
 if [ $is_public -eq 1 ]
@@ -176,5 +189,12 @@ fi
 #=================================================
 # RELOAD NGINX
 #=================================================
+ynh_print_info "Reloading nginx web server..."
 
 systemctl reload nginx
+
+#=================================================
+# END OF SCRIPT
+#=================================================
+
+ynh_print_info "Installation of $app completed"
diff --git a/scripts/remove b/scripts/remove
index 37d1c83..f29081d 100644
--- a/scripts/remove
+++ b/scripts/remove
@@ -12,6 +12,7 @@ source /usr/share/yunohost/helpers
 #=================================================
 # LOAD SETTINGS
 #=================================================
+ynh_print_info "Loading installation settings..."
 
 app=$YNH_APP_INSTANCE_NAME
 
@@ -25,6 +26,7 @@ final_path=$(ynh_app_setting_get $app final_path)
 #=================================================
 # REMOVE THE MYSQL DATABASE
 #=================================================
+ynh_print_info "Removing the MySQL database"
 
 # Remove a database if it exists, along with the associated user
 ynh_mysql_remove_db $db_user $db_name
@@ -32,6 +34,7 @@ ynh_mysql_remove_db $db_user $db_name
 #=================================================
 # REMOVE APP MAIN DIR
 #=================================================
+ynh_print_info "Removing app main directory"
 
 # Remove the app directory securely
 ynh_secure_remove "$final_path"
@@ -39,6 +42,7 @@ ynh_secure_remove "$final_path"
 #=================================================
 # REMOVE NGINX CONFIGURATION
 #=================================================
+ynh_print_info "Removing nginx web server configuration"
 
 # Remove the dedicated nginx config
 ynh_remove_nginx_config
@@ -46,6 +50,7 @@ ynh_remove_nginx_config
 #=================================================
 # REMOVE PHP-FPM CONFIGURATION
 #=================================================
+ynh_print_info "Removing php-fpm configuration"
 
 # Remove the dedicated php-fpm config
 ynh_remove_fpm_config
@@ -67,6 +72,13 @@ ynh_secure_remove "/var/log/$app/"
 #=================================================
 # REMOVE DEDICATED USER
 #=================================================
+ynh_print_info "Removing the dedicated system user"
 
 # Delete a system user
 ynh_system_user_delete $app
+
+#=================================================
+# END OF SCRIPT
+#=================================================
+
+ynh_print_info "Removal of $app completed"
diff --git a/scripts/restore b/scripts/restore
index 3ab1d08..8a3944c 100644
--- a/scripts/restore
+++ b/scripts/restore
@@ -19,6 +19,7 @@ ynh_abort_if_errors
 #=================================================
 # LOAD SETTINGS
 #=================================================
+ynh_print_info "Loading settings..."
 
 app=$YNH_APP_INSTANCE_NAME
 
@@ -30,6 +31,7 @@ db_name=$(ynh_app_setting_get $app db_name)
 #=================================================
 # CHECK IF THE APP CAN BE RESTORED
 #=================================================
+ynh_print_info "Validating restoration parameters..."
 
 ynh_webpath_available $domain $path_url \
 	|| ynh_die "Path not available: ${domain}${path_url}"
@@ -47,20 +49,14 @@ ynh_restore_file "/etc/nginx/conf.d/$domain.d/$app.conf"
 #=================================================
 # RESTORE THE APP MAIN DIR
 #=================================================
+ynh_print_info "Restoring the app main directory..."
 
 ynh_restore_file "$final_path"
 
-#=================================================
-# RESTORE THE MYSQL DATABASE
-#=================================================
-
-db_pwd=$(ynh_app_setting_get $app mysqlpwd)
-ynh_mysql_setup_db $db_name $db_name $db_pwd
-ynh_mysql_connect_as $db_name $db_pwd $db_name < ./db.sql
-
 #=================================================
 # RECREATE THE DEDICATED USER
 #=================================================
+ynh_print_info "Recreating the dedicated system user..."
 
 # Create the dedicated user (if not existing)
 ynh_system_user_create $app
@@ -79,6 +75,14 @@ chown -R $app: $final_path/var
 #=================================================
 
 ynh_restore_file "/etc/php7.0/fpm/pool.d/$app.conf"
+#=================================================
+# RESTORE THE MYSQL DATABASE
+#=================================================
+ynh_print_info "Restoring the MySQL database..."
+
+db_pwd=$(ynh_app_setting_get $app mysqlpwd)
+ynh_mysql_setup_db $db_name $db_name $db_pwd
+ynh_mysql_connect_as $db_name $db_pwd $db_name < ./db.sql
 
 #=================================================
 # RESTORE THE CRON FILE
@@ -91,6 +95,13 @@ ynh_restore_file "/etc/cron.d/$app"
 #=================================================
 # RELOAD NGINX AND PHP-FPM
 #=================================================
+ynh_print_info "Reloading nginx web server and php-fpm..."
 
 systemctl reload php5-fpm
 systemctl reload nginx
+
+#=================================================
+# END OF SCRIPT
+#=================================================
+
+ynh_print_info "Restoration completed for $app"
diff --git a/scripts/upgrade b/scripts/upgrade
index ce8c74d..a4632d4 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -12,6 +12,7 @@ source /usr/share/yunohost/helpers
 #=================================================
 # LOAD SETTINGS
 #=================================================
+ynh_print_info "Loading installation settings..."
 
 app=$YNH_APP_INSTANCE_NAME
 
@@ -28,6 +29,7 @@ upstream_version=$(ynh_app_upstream_version)
 #=================================================
 # ENSURE DOWNWARD COMPATIBILITY
 #=================================================
+ynh_print_info "Ensuring downward compatibility..."
 
 # Fix is_public as a boolean value
 if [ "$is_public" = "Yes" ]; then
@@ -53,6 +55,7 @@ fi
 #=================================================
 # BACKUP BEFORE UPGRADE THEN ACTIVE TRAP
 #=================================================
+ynh_print_info "Backing up the app before upgrading (may take a while)..."
 
 # Backup the current version of the app
 ynh_backup_before_upgrade
@@ -75,6 +78,7 @@ path_url=$(ynh_normalize_url_path $path_url)
 #=================================================
 # DOWNLOAD, CHECK AND UNPACK SOURCE
 #=================================================
+ynh_print_info "Upgrading source files..."
 
 # Download, check integrity, uncompress and patch the source from app.src
 ynh_setup_source "$final_path"
@@ -82,6 +86,7 @@ ynh_setup_source "$final_path"
 #=================================================
 # NGINX CONFIGURATION
 #=================================================
+ynh_print_info "Upgrading nginx web server configuration..."
 
 # Create a dedicated nginx config
 ynh_add_nginx_config
@@ -89,13 +94,15 @@ ynh_add_nginx_config
 #=================================================
 # CREATE DEDICATED USER
 #=================================================
+ynh_print_info "Making sure dedicated system user exists..."
 
-# Create a system user
+# Create a dedicated user (if not existing)
 ynh_system_user_create $app
 
 #=================================================
 # PHP-FPM CONFIGURATION
 #=================================================
+ynh_print_info "Upgrading php-fpm configuration..."
 
 # Create a dedicated php-fpm config
 ynh_add_fpm_config
@@ -126,7 +133,7 @@ fi
 # SECURE FILES AND DIRECTORIES
 #=================================================
 
-# Set permissions to app files
+# Set permissions on app files
 chown -R root: $final_path
 
 # Folders that needs to be writable by cheky
@@ -136,6 +143,7 @@ chown -R $app: $final_path/static/media/annonce   # needed for "Mes annonces sau
 #=================================================
 # SETUP SSOWAT
 #=================================================
+ynh_print_info "Upgrading SSOwat configuration..."
 
 # Make app public if necessary
 if [ $is_public -eq 1 ]
@@ -147,5 +155,12 @@ fi
 #=================================================
 # RELOAD NGINX
 #=================================================
+ynh_print_info "Reloading nginx web server..."
 
 systemctl reload nginx
+
+#=================================================
+# END OF SCRIPT
+#=================================================
+
+ynh_print_info "Upgrade of $app completed"

From d9f2b2a7ecae44b561bb4cf962dcd4c045ab8edc Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 4 Mar 2019 23:52:56 +0100
Subject: [PATCH 03/19] Drop jessie support and use php7

---
 conf/php-fpm.conf | 63 +++++++++++++++++++++++++++++++++--------------
 manifest.json     |  4 +--
 scripts/restore   |  2 +-
 3 files changed, 48 insertions(+), 21 deletions(-)

diff --git a/conf/php-fpm.conf b/conf/php-fpm.conf
index 58ab5e2..9018c9d 100644
--- a/conf/php-fpm.conf
+++ b/conf/php-fpm.conf
@@ -1,10 +1,11 @@
 ; Start a new pool named 'www'.
-; the variable $pool can we used in any directive and will be replaced by the
+; the variable $pool can be used in any directive and will be replaced by the
 ; pool name ('www' here)
 [__NAMETOCHANGE__]
 
 ; Per pool prefix
 ; It only applies on the following directives:
+; - 'access.log'
 ; - 'slowlog'
 ; - 'listen' (unixsocket)
 ; - 'chroot'
@@ -24,17 +25,19 @@ group = __USER__
 
 ; The address on which to accept FastCGI requests.
 ; Valid syntaxes are:
-;   'ip.add.re.ss:port'    - to listen on a TCP socket to a specific address on
+;   'ip.add.re.ss:port'    - to listen on a TCP socket to a specific IPv4 address on
 ;                            a specific port;
-;   'port'                 - to listen on a TCP socket to all addresses on a
-;                            specific port;
+;   '[ip:6:addr:ess]:port' - to listen on a TCP socket to a specific IPv6 address on
+;                            a specific port;
+;   'port'                 - to listen on a TCP socket to all addresses
+;                            (IPv6 and IPv4-mapped) on a specific port;
 ;   '/path/to/unix/socket' - to listen on a unix socket.
 ; Note: This value is mandatory.
-listen = /var/run/php5-fpm-__NAMETOCHANGE__.sock
+listen = /var/run/php/php7.0-fpm-__NAMETOCHANGE__.sock
 
-; Set listen(2) backlog. A value of '-1' means unlimited.
-; Default Value: 128 (-1 on FreeBSD and OpenBSD)
-;listen.backlog = 128
+; Set listen(2) backlog.
+; Default Value: 511 (-1 on FreeBSD and OpenBSD)
+;listen.backlog = 511
 
 ; Set permissions for unix socket, if one is used. In Linux, read/write
 ; permissions must be set in order to allow connections from a web server. Many
@@ -44,8 +47,13 @@ listen = /var/run/php5-fpm-__NAMETOCHANGE__.sock
 listen.owner = www-data
 listen.group = www-data
 ;listen.mode = 0660
- 
-; List of ipv4 addresses of FastCGI clients which are allowed to connect.
+; When POSIX Access Control Lists are supported you can set them using
+; these options, value is a comma separated list of user/group names.
+; When set, listen.owner and listen.group are ignored
+;listen.acl_users =
+;listen.acl_groups =
+
+; List of addresses (IPv4/IPv6) of FastCGI clients which are allowed to connect.
 ; Equivalent to the FCGI_WEB_SERVER_ADDRS environment variable in the original
 ; PHP FCGI (5.2.2+). Makes sense only with a tcp listening socket. Each address
 ; must be separated by a comma. If this value is left blank, connections will be
@@ -59,7 +67,13 @@ listen.group = www-data
 ;       - The pool processes will inherit the master process priority
 ;         unless it specified otherwise
 ; Default Value: no set
-; priority = -19
+; process.priority = -19
+
+; Set the process dumpable flag (PR_SET_DUMPABLE prctl) even if the process user
+; or group is differrent than the master process user. It allows to create process
+; core dump and ptrace the process for the pool user.
+; Default Value: no
+; process.dumpable = yes
 
 ; Choose how the process manager will control the number of child processes.
 ; Possible Values:
@@ -96,7 +110,7 @@ pm = dynamic
 ; forget to tweak pm.* to fit your needs.
 ; Note: Used when pm is set to 'static', 'dynamic' or 'ondemand'
 ; Note: This value is mandatory.
-pm.max_children = 10
+pm.max_children = 5
 
 ; The number of child processes created on startup.
 ; Note: Used only when pm is set to 'dynamic'
@@ -122,7 +136,7 @@ pm.max_spare_servers = 3
 ; This can be useful to work around memory leaks in 3rd party libraries. For
 ; endless request processing specify '0'. Equivalent to PHP_FCGI_MAX_REQUESTS.
 ; Default Value: 0
-pm.max_requests = 500
+;pm.max_requests = 500
 
 ; The URI to view the FPM status page. If this value is not set, no URI will be
 ; recognized as a status page. It shows the following informations:
@@ -215,7 +229,7 @@ pm.max_requests = 500
 ;   last request memory:  0
 ;
 ; Note: There is a real-time FPM status monitoring sample web page available
-;       It's available in: ${prefix}/share/fpm/status.html
+;       It's available in: /usr/share/php/7.0/fpm/status.html
 ;
 ; Note: The value must start with a leading slash (/). The value can be
 ;       anything, but it may not be a good idea to use the .php extension or it
@@ -275,7 +289,7 @@ pm.max_requests = 500
 ;      - %{megabytes}M
 ;      - %{mega}M
 ;  %n: pool name
-;  %o: ouput header
+;  %o: output header
 ;      it must be associated with embraces to specify the name of the header:
 ;      - %{Content-Type}o
 ;      - %{X-Powered-By}o
@@ -291,9 +305,13 @@ pm.max_requests = 500
 ;  %t: server time the request was received
 ;      it can accept a strftime(3) format:
 ;      %d/%b/%Y:%H:%M:%S %z (default)
+;      The strftime(3) format must be encapsuled in a %{<strftime_format>}t tag
+;      e.g. for a ISO8601 formatted timestring, use: %{%Y-%m-%dT%H:%M:%S%z}t
 ;  %T: time the log has been written (the request has finished)
 ;      it can accept a strftime(3) format:
 ;      %d/%b/%Y:%H:%M:%S %z (default)
+;      The strftime(3) format must be encapsuled in a %{<strftime_format>}t tag
+;      e.g. for a ISO8601 formatted timestring, use: %{%Y-%m-%dT%H:%M:%S%z}t
 ;  %u: remote user
 ;
 ; Default: "%R - %u %t \"%m %r\" %s"
@@ -347,15 +365,24 @@ chdir = __FINALPATH__
 ; Note: on highloaded environement, this can cause some delay in the page
 ; process time (several ms).
 ; Default Value: no
-catch_workers_output = yes
+;catch_workers_output = yes
+
+; Clear environment in FPM workers
+; Prevents arbitrary environment variables from reaching FPM worker processes
+; by clearing the environment in workers before env vars specified in this
+; pool configuration are added.
+; Setting to "no" will make all environment variables available to PHP code
+; via getenv(), $_ENV and $_SERVER.
+; Default Value: yes
+;clear_env = no
 
 ; Limits the extensions of the main script FPM will allow to parse. This can
 ; prevent configuration mistakes on the web server side. You should only limit
 ; FPM to .php extensions to prevent malicious users to use other extensions to
-; exectute php code.
+; execute php code.
 ; Note: set an empty value to allow all extensions.
 ; Default Value: .php
-;security.limit_extensions = .php .php3 .php4 .php5
+;security.limit_extensions = .php .php3 .php4 .php5 .php7
  
 ; Pass environment variables like LD_LIBRARY_PATH. All $VARIABLEs are taken from
 ; the current environment.
diff --git a/manifest.json b/manifest.json
index c41312f..2012d02 100644
--- a/manifest.json
+++ b/manifest.json
@@ -27,12 +27,12 @@
         }
     ],
     "requirements": {
-        "yunohost": ">> 2.7.14"
+        "yunohost": ">= 3.4"
     },
     "multi_instance": true,
     "services": [
         "nginx",
-        "php5-fpm",
+        "php7.0-fpm",
         "mysql"
     ],
     "arguments": {
diff --git a/scripts/restore b/scripts/restore
index 8a3944c..1c6fa66 100644
--- a/scripts/restore
+++ b/scripts/restore
@@ -97,7 +97,7 @@ ynh_restore_file "/etc/cron.d/$app"
 #=================================================
 ynh_print_info "Reloading nginx web server and php-fpm..."
 
-systemctl reload php5-fpm
+systemctl reload php7.0-fpm
 systemctl reload nginx
 
 #=================================================

From ecf6697f20d3dcd42df030535237eb92b180c709 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 4 Mar 2019 23:53:39 +0100
Subject: [PATCH 04/19] [enh] misc enhancements

---
 scripts/_common.sh | 23 ++++++++++-------------
 scripts/install    | 22 ++++++++++------------
 2 files changed, 20 insertions(+), 25 deletions(-)

diff --git a/scripts/_common.sh b/scripts/_common.sh
index 64e634b..4718616 100644
--- a/scripts/_common.sh
+++ b/scripts/_common.sh
@@ -1,23 +1,17 @@
 #!/bin/bash
 
-# ============= FUTURE YUNOHOST HELPER =============
-# Delete a file checksum from the app settings
-#
-# $app should be defined when calling this helper
-#
-# usage: ynh_remove_file_checksum file
-# | arg: file - The file for which the checksum will be deleted
-ynh_delete_file_checksum () {
-	local checksum_setting_name=checksum_${1//[\/ ]/_}	# Replace all '/' and ' ' by '_'
-	ynh_app_setting_delete $app $checksum_setting_name
-}
+#=================================================
+# PERSONAL HELPERS
+#=================================================
 
+#=================================================
+# EXPERIMENTAL HELPERS
+#=================================================
 
-
-### Experimental helpers
 # Taken from https://github.com/YunoHost-Apps/Experimental_helpers/blob/master/ynh_read_manifest/ynh_read_manifest_2#L14-L28
 # Idea from  https://forum.yunohost.org/t/upgrade-script-how-to-modify-parameter-inside-configuration-file/5352/2
 
+
 # Read the value of a key in a ynh manifest file
 #
 # usage: ynh_read_manifest manifest key
@@ -45,3 +39,6 @@ ynh_app_upstream_version () {
     echo "${version_key/~ynh*/}"
 }
 
+#=================================================
+# FUTURE OFFICIAL HELPERS
+#=================================================
diff --git a/scripts/install b/scripts/install
index b0e6329..b31177e 100644
--- a/scripts/install
+++ b/scripts/install
@@ -25,16 +25,16 @@ path_url=$YNH_APP_ARG_PATH
 password=$YNH_APP_ARG_PASSWORD
 is_public=$YNH_APP_ARG_IS_PUBLIC
 
-# This is a multi-instance app, meaning it can be installed several times independently
-# The id of the app as stated in the manifest is available as $YNH_APP_ID
-# The instance number is available as $YNH_APP_INSTANCE_NUMBER (equals "1", "2", ...)
-# The app instance name is available as $YNH_APP_INSTANCE_NAME
-#    - the first time the app is installed, YNH_APP_INSTANCE_NAME = ynhexample
-#    - the second time the app is installed, YNH_APP_INSTANCE_NAME = ynhexample__2
-#    - ynhexample__{N} for the subsequent installations, with N=3,4, ...
-# The app instance name is probably what you are interested the most, since this is
-# guaranteed to be unique. This is a good unique identifier to define installation path,
-# db names, ...
+### If it's a multi-instance app, meaning it can be installed several times independently
+### The id of the app as stated in the manifest is available as $YNH_APP_ID
+### The instance number is available as $YNH_APP_INSTANCE_NUMBER (equals "1", "2", ...)
+### The app instance name is available as $YNH_APP_INSTANCE_NAME
+###    - the first time the app is installed, YNH_APP_INSTANCE_NAME = ynhexample
+###    - the second time the app is installed, YNH_APP_INSTANCE_NAME = ynhexample__2
+###    - ynhexample__{N} for the subsequent installations, with N=3,4, ...
+### The app instance name is probably what interests you most, since this is
+### guaranteed to be unique. This is a good unique identifier to define installation path,
+### db names, ...
 app=$YNH_APP_INSTANCE_NAME
 
 #=================================================
@@ -48,8 +48,6 @@ test ! -e "$final_path" || ynh_die "This path already contains a folder"
 # Normalize the url path syntax
 path_url=$(ynh_normalize_url_path $path_url)
 
-# Check web path availability
-ynh_webpath_available $domain $path_url
 # Register (book) web path
 ynh_webpath_register $app $domain $path_url
 

From a4bb81816766c5aaff742edeebec3aca49cd8cab Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 4 Mar 2019 23:54:28 +0100
Subject: [PATCH 05/19] [doc] Upgrade to version 4.4~ynh4

---
 README.md     | 5 +++++
 manifest.json | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 453f303..fe6c060 100644
--- a/README.md
+++ b/README.md
@@ -56,6 +56,11 @@ sudo yunohost app upgrade cheky --url https://github.com/YunoHost-Apps/cheky_ynh
 
 ## Historique des versions
 
+* 4.4~ynh4 (4 March 2019)
+  * removed: Debian Jessie support is dropped in favor to Stretch only (Yunohost version > 3)
+  * changed: follow lastest rules from <https://github.com/YunoHost/example_ynh>
+  * fixed: remove old php file <https://github.com/YunoHost/example_ynh/pull/52>
+  * changed: increment YunoHost package version to `ynh4`
 * 24 Feb 2019 [#16](https://github.com/YunoHost-Apps/cheky_ynh/pull/16) - Mise à jour vers la version 4.4
 * 24 Sep 2018 [#11](https://github.com/YunoHost-Apps/cheky_ynh/pull/11) - Mise à jour vers la version 4.3.5
 * 14 Sep 2018 [10](https://github.com/YunoHost-Apps/cheky_ynh/pull/10) - Correction images dans "annonces sauvegardées"
diff --git a/manifest.json b/manifest.json
index 2012d02..558c8b2 100644
--- a/manifest.json
+++ b/manifest.json
@@ -6,7 +6,7 @@
         "en": "Cheky monitors French classifieds websites LeBonCoin and SeLoger.",
         "fr": "Cheky surveille pour vous LeBonCoin et SeLoger."
     },
-    "version": "4.4~ynh3",
+    "version": "4.4~ynh4",
     "url": "https://www.cheky.net",
     "license": "GPL-3.0-or-later",
     "maintainer": {

From 3ae7fd9d748f8ed2435ac997a1d7f21a2840f265 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Mon, 11 Mar 2019 22:19:33 +0100
Subject: [PATCH 06/19] Upgrade to version 4.4.1

---
 README.md     | 4 +++-
 conf/app.src  | 4 ++--
 manifest.json | 2 +-
 3 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/README.md b/README.md
index fe6c060..709bd32 100644
--- a/README.md
+++ b/README.md
@@ -6,7 +6,7 @@
 > *This package allow you to install cheky quickly and simply on a YunoHost server.
 If you don't have YunoHost, please see [here](https://yunohost.org/#/install) to know how to install and enjoy it.*
 
-**Shipped version:** 4.4
+**Shipped version:** 4.4.1
 
 ## Features
 
@@ -56,6 +56,8 @@ sudo yunohost app upgrade cheky --url https://github.com/YunoHost-Apps/cheky_ynh
 
 ## Historique des versions
 
+* 4.4.1~ynh4 (11 March 2019)
+  * changed: upgrade Cheky to 4.4.1
 * 4.4~ynh4 (4 March 2019)
   * removed: Debian Jessie support is dropped in favor to Stretch only (Yunohost version > 3)
   * changed: follow lastest rules from <https://github.com/YunoHost/example_ynh>
diff --git a/conf/app.src b/conf/app.src
index 639f7eb..73b3426 100644
--- a/conf/app.src
+++ b/conf/app.src
@@ -1,5 +1,5 @@
-SOURCE_URL=https://github.com/Blount/Cheky/archive/4.4.tar.gz
-SOURCE_SUM=1916d6196d48a838ea55d518099f75d941e84125c6a3c00a3943ad3bb0a2714d
+SOURCE_URL=https://github.com/Blount/Cheky/archive/4.4.1.tar.gz
+SOURCE_SUM=4142430d17229332efca353cc95a993d85650af0a53155055847b4d7b9e9650b
 SOURCE_SUM_PRG=sha256sum
 SOURCE_FORMAT=tar.gz
 SOURCE_IN_SUBDIR=true
diff --git a/manifest.json b/manifest.json
index 558c8b2..bee0e8e 100644
--- a/manifest.json
+++ b/manifest.json
@@ -6,7 +6,7 @@
         "en": "Cheky monitors French classifieds websites LeBonCoin and SeLoger.",
         "fr": "Cheky surveille pour vous LeBonCoin et SeLoger."
     },
-    "version": "4.4~ynh4",
+    "version": "4.4.1~ynh4",
     "url": "https://www.cheky.net",
     "license": "GPL-3.0-or-later",
     "maintainer": {

From 45d1dea0a820995292c4bd68b4ec3763c0b5cbb0 Mon Sep 17 00:00:00 2001
From: md-clone <md-clone@users.noreply.github.com>
Date: Mon, 25 Mar 2019 09:34:56 +0100
Subject: [PATCH 07/19] Update backup

php-fpm configuration path fix
---
 scripts/backup | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/backup b/scripts/backup
index 30db841..bea564c 100644
--- a/scripts/backup
+++ b/scripts/backup
@@ -48,7 +48,7 @@ ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 #=================================================
 ynh_print_info "Backing up php-fpm configuration..."
 
-ynh_backup "/etc/php7.0/fpm/pool.d/$app.conf"
+ynh_backup "/etc/php/7.0/fpm/pool.d/$app.conf"
 
 #=================================================
 # BACKUP THE MYSQL DATABASE

From 2b4015d12e63444dc892c3caf816a0d25696154a Mon Sep 17 00:00:00 2001
From: md-clone <md-clone@users.noreply.github.com>
Date: Mon, 25 Mar 2019 09:35:46 +0100
Subject: [PATCH 08/19] Update restore

php-fpm configuration path fix
---
 scripts/restore | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/restore b/scripts/restore
index 1c6fa66..9e8d8ca 100644
--- a/scripts/restore
+++ b/scripts/restore
@@ -74,7 +74,7 @@ chown -R $app: $final_path/var
 # RESTORE THE PHP-FPM CONFIGURATION
 #=================================================
 
-ynh_restore_file "/etc/php7.0/fpm/pool.d/$app.conf"
+ynh_restore_file "/etc/php/7.0/fpm/pool.d/$app.conf"
 #=================================================
 # RESTORE THE MYSQL DATABASE
 #=================================================

From 11c21bc7f9774815808deb97f1c7af6f9e6043e4 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Wed, 8 May 2019 14:33:56 +0200
Subject: [PATCH 09/19] [enh] add test upgrade from old commit

Untested. "Should" work
---
 check_process | 1 +
 1 file changed, 1 insertion(+)

diff --git a/check_process b/check_process
index 4190fc4..5d0c888 100644
--- a/check_process
+++ b/check_process
@@ -20,6 +20,7 @@
 		setup_private=1
 		setup_public=1
 		upgrade=1
+		upgrade=1 from_commit=a6b2615101887e3235aab0b95607eb1b58507a4c
 		backup_restore=1
 		multi_instance=1
 		incorrect_path=1

From 00bcdc08a6f0d2756bbe58e81e5edcd276bc81b2 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Wed, 8 May 2019 14:37:38 +0200
Subject: [PATCH 10/19] Remove broken automated upgrade method

This method does not fit anymore with last version. Must be changed
---
 scripts/_common.sh | 26 --------------------------
 scripts/upgrade    |  7 -------
 2 files changed, 33 deletions(-)

diff --git a/scripts/_common.sh b/scripts/_common.sh
index 4718616..f0851b0 100644
--- a/scripts/_common.sh
+++ b/scripts/_common.sh
@@ -11,32 +11,6 @@
 # Taken from https://github.com/YunoHost-Apps/Experimental_helpers/blob/master/ynh_read_manifest/ynh_read_manifest_2#L14-L28
 # Idea from  https://forum.yunohost.org/t/upgrade-script-how-to-modify-parameter-inside-configuration-file/5352/2
 
-
-# Read the value of a key in a ynh manifest file
-#
-# usage: ynh_read_manifest manifest key
-# | arg: manifest - Path of the manifest to read
-# | arg: key - Name of the key to find
-ynh_read_manifest () {
-	manifest="$1"
-	key="$2"
-	python3 -c "import sys, json;print(json.load(open('$manifest', encoding='utf-8'))['$key'])"
-}
-
-# Read the upstream version from the manifest
-# The version number in the manifest is defined by <upstreamversion>~ynh<packageversion>
-# For example : 4.3-2~ynh3
-# This include the number before ~ynh
-# In the last example it return 4.3-2
-#
-# usage: ynh_app_upstream_version
-ynh_app_upstream_version () {
-    manifest_path="../manifest.json"
-    if [ ! -e "$manifest_path" ]; then
-        manifest_path="../settings/manifest.json"	# Into the restore script, the manifest is not at the same place
-    fi
-    version_key=$(ynh_read_manifest "$manifest_path" "version")
-    echo "${version_key/~ynh*/}"
 }
 
 #=================================================
diff --git a/scripts/upgrade b/scripts/upgrade
index a4632d4..f35a96b 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -23,9 +23,6 @@ is_public=$(ynh_app_setting_get $app is_public)
 final_path=$(ynh_app_setting_get $app final_path)
 db_name=$(ynh_app_setting_get $app db_name)
 
-# Exerimental helper, see "_common.sh"
-upstream_version=$(ynh_app_upstream_version)
-
 #=================================================
 # ENSURE DOWNWARD COMPATIBILITY
 #=================================================
@@ -122,10 +119,6 @@ ynh_add_fpm_config
 # If missing, Cheky is stuck in an "upgrade" state and the config file
 # needs to be edited with latest version
 # Also avoid to make file "$final_path/version.php" writable
-if [ -f "$final_path/var/config.ini" ]; then
-	# Change the existing version by the new one taken from the app manifest
-	ynh_replace_string "^version =.*" "version = \"$upstream_version\"" "$final_path/var/config.ini"
-fi
 
 #=================================================
 # GENERIC FINALIZATION

From 36e152c159587b66a58a4853826735b158b99c4a Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Wed, 8 May 2019 14:40:34 +0200
Subject: [PATCH 11/19] Add helper 'exec_as' and use it

---
 scripts/_common.sh | 13 +++++++++++--
 scripts/upgrade    |  1 +
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/scripts/_common.sh b/scripts/_common.sh
index f0851b0..c85fc0e 100644
--- a/scripts/_common.sh
+++ b/scripts/_common.sh
@@ -8,9 +8,18 @@
 # EXPERIMENTAL HELPERS
 #=================================================
 
-# Taken from https://github.com/YunoHost-Apps/Experimental_helpers/blob/master/ynh_read_manifest/ynh_read_manifest_2#L14-L28
-# Idea from  https://forum.yunohost.org/t/upgrade-script-how-to-modify-parameter-inside-configuration-file/5352/2
+# Execute a command as another user
+# usage: exec_as USER COMMAND [ARG ...]
+# Source : https://github.com/YunoHost-Apps/Experimental_helpers/tree/master/ynh_exec_as
+exec_as() {
+  local USER=$1
+  shift 1
 
+  if [[ $USER = $(whoami) ]]; then
+    eval "$@"
+  else
+    sudo -u "$USER" "$@"
+  fi
 }
 
 #=================================================
diff --git a/scripts/upgrade b/scripts/upgrade
index f35a96b..4bf48eb 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -120,6 +120,7 @@ ynh_add_fpm_config
 # needs to be edited with latest version
 # Also avoid to make file "$final_path/version.php" writable
 
+#exec_as "$app" "/usr/bin/php $YNH_CWD/upgrade_cheky.php $final_path"
 #=================================================
 # GENERIC FINALIZATION
 #=================================================

From de7e7522b298ccc7495d626824fd840e313f90e1 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Wed, 8 May 2019 14:41:10 +0200
Subject: [PATCH 12/19] WIP - disable exit on error for debuging purpose

---
 scripts/upgrade | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/scripts/upgrade b/scripts/upgrade
index 4bf48eb..98404ed 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -54,14 +54,14 @@ fi
 #=================================================
 ynh_print_info "Backing up the app before upgrading (may take a while)..."
 
-# Backup the current version of the app
-ynh_backup_before_upgrade
-ynh_clean_setup () {
-	# restore it if the upgrade fails
-	ynh_restore_upgradebackup
-}
-# Exit if an error occurs during the execution of the script
-ynh_abort_if_errors
+## Backup the current version of the app
+#ynh_backup_before_upgrade
+#ynh_clean_setup () {
+#	# restore it if the upgrade fails
+#	ynh_restore_upgradebackup
+#}
+## Exit if an error occurs during the execution of the script
+#ynh_abort_if_errors
 
 #=================================================
 # CHECK THE PATH

From 5af9dc26f51f48d2940739b293bf0e02c29004c2 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Wed, 8 May 2019 14:42:53 +0200
Subject: [PATCH 13/19] WIP - rework automated upgrade method

Does not work at the moment.
---
 scripts/upgrade           | 18 +++++++++++++++++-
 scripts/upgrade_cheky.php | 37 +++++++++++++++++++++++++++++++++++++
 2 files changed, 54 insertions(+), 1 deletion(-)
 create mode 100644 scripts/upgrade_cheky.php

diff --git a/scripts/upgrade b/scripts/upgrade
index 98404ed..787aebf 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -109,7 +109,8 @@ ynh_add_fpm_config
 #=================================================
 # ...
 #=================================================
-#todo: part needed ?
+
+ynh_print_info "Running specific upgrade..."
 # Verify the checksum and backup the file if it's different
 #ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE"
 # Recalculate and store the config file checksum into the app settings
@@ -120,7 +121,22 @@ ynh_add_fpm_config
 # needs to be edited with latest version
 # Also avoid to make file "$final_path/version.php" writable
 
+# Give full access to "$app" so php script can do "its own magic stuff"
+chown -R $app: $final_path
+
 #exec_as "$app" "/usr/bin/php $YNH_CWD/upgrade_cheky.php $final_path"
+sudo -u "$app" "/usr/bin/php $YNH_CWD/upgrade_cheky.php $final_path"
+
+#=================================================
+# CONFIGURE HOOKS
+#=================================================
+### TODO: Better to use a hook like "post_app_upgrade" ?
+#ynh_print_info "Running post upgrade recipe..."
+
+#ynh_replace_string "__FINALPATH__" "$final_path" "../hooks/post_app_install"
+#ynh_replace_string "__FINALPATH__" "$final_path" "../hooks/post_app_remove"
+
+
 #=================================================
 # GENERIC FINALIZATION
 #=================================================
diff --git a/scripts/upgrade_cheky.php b/scripts/upgrade_cheky.php
new file mode 100644
index 0000000..645af4d
--- /dev/null
+++ b/scripts/upgrade_cheky.php
@@ -0,0 +1,37 @@
+<?php
+
+// source: https://forum.cheky.net/mise-a-jour-de-cheky-en-ligne-de-commande-t659-p1.html#p2624
+
+//$root_path = "__FINALPATH__";
+$root_path = $_SERVER["argv"][1];
+if (empty($root_path) || !is_dir($root_path)) {
+    fwrite(STDERR, "Chemin manquant dans hook : post_app_upgrade");
+    exit(1);
+}
+
+
+require $root_path."/bootstrap.php";
+
+$storageType = $config->get("storage", "type", "files");
+if ($storageType == "db") {
+    $userStorage = new \App\Storage\Db\User($dbConnection);
+} else {
+    $userStorage = new \App\Storage\File\User(DOCUMENT_ROOT."/var/users.db");
+}
+
+//require $root_path."/bootstrap.php";
+
+$_POST = array(
+    "upgrade" => 1,
+);
+
+require $root_path."/app/admin/scripts/upgrade.php";
+
+// S'il y a des erreurs, on les écrit dans STDERR et on quitte
+// avec un code erreur.
+#if (!empty($errors)) {
+#    fwrite(STDERR, str_replace(
+#        array("<br>", "<br />"), "", implode("\n", $errors)
+#    );
+#    exit(1);
+#}

From c368ad4bfee566ecd4ba03506dff666041de2d5d Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 00:27:31 +0200
Subject: [PATCH 14/19] fix automated upgrade

---
 scripts/upgrade                                  | 15 +++------------
 ...rade_cheky.php => yunohost_upgrade_cheky.php} | 16 +++++++---------
 2 files changed, 10 insertions(+), 21 deletions(-)
 rename scripts/{upgrade_cheky.php => yunohost_upgrade_cheky.php} (77%)

diff --git a/scripts/upgrade b/scripts/upgrade
index 787aebf..92642dd 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -124,18 +124,9 @@ ynh_print_info "Running specific upgrade..."
 # Give full access to "$app" so php script can do "its own magic stuff"
 chown -R $app: $final_path
 
-#exec_as "$app" "/usr/bin/php $YNH_CWD/upgrade_cheky.php $final_path"
-sudo -u "$app" "/usr/bin/php $YNH_CWD/upgrade_cheky.php $final_path"
-
-#=================================================
-# CONFIGURE HOOKS
-#=================================================
-### TODO: Better to use a hook like "post_app_upgrade" ?
-#ynh_print_info "Running post upgrade recipe..."
-
-#ynh_replace_string "__FINALPATH__" "$final_path" "../hooks/post_app_install"
-#ynh_replace_string "__FINALPATH__" "$final_path" "../hooks/post_app_remove"
-
+# Copy Yunohost custom script to cheky folder and launch it
+cp yunohost_upgrade_cheky.php $final_path/others/update/
+exec_as "$app" /usr/bin/php $final_path/others/update/yunohost_upgrade_cheky.php $final_path
 
 #=================================================
 # GENERIC FINALIZATION
diff --git a/scripts/upgrade_cheky.php b/scripts/yunohost_upgrade_cheky.php
similarity index 77%
rename from scripts/upgrade_cheky.php
rename to scripts/yunohost_upgrade_cheky.php
index 645af4d..99b1eed 100644
--- a/scripts/upgrade_cheky.php
+++ b/scripts/yunohost_upgrade_cheky.php
@@ -1,5 +1,5 @@
 <?php
-
+// Script d'upgrade custom pour automatiser la MAJ sans action de l'utilisateur
 // source: https://forum.cheky.net/mise-a-jour-de-cheky-en-ligne-de-commande-t659-p1.html#p2624
 
 //$root_path = "__FINALPATH__";
@@ -19,8 +19,6 @@ if ($storageType == "db") {
     $userStorage = new \App\Storage\File\User(DOCUMENT_ROOT."/var/users.db");
 }
 
-//require $root_path."/bootstrap.php";
-
 $_POST = array(
     "upgrade" => 1,
 );
@@ -29,9 +27,9 @@ require $root_path."/app/admin/scripts/upgrade.php";
 
 // S'il y a des erreurs, on les écrit dans STDERR et on quitte
 // avec un code erreur.
-#if (!empty($errors)) {
-#    fwrite(STDERR, str_replace(
-#        array("<br>", "<br />"), "", implode("\n", $errors)
-#    );
-#    exit(1);
-#}
+if (!empty($errors)) {
+    fwrite(STDERR, str_replace(
+        array("<br>", "<br />"), "", implode("\n", $errors)
+    ));
+    exit(1);
+}

From d27beaf0cfbac05a3004bb0ea62022d7f3d5a434 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 00:27:56 +0200
Subject: [PATCH 15/19] [fix] backup script

---
 scripts/backup | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/backup b/scripts/backup
index 30db841..bea564c 100644
--- a/scripts/backup
+++ b/scripts/backup
@@ -48,7 +48,7 @@ ynh_backup "/etc/nginx/conf.d/$domain.d/$app.conf"
 #=================================================
 ynh_print_info "Backing up php-fpm configuration..."
 
-ynh_backup "/etc/php7.0/fpm/pool.d/$app.conf"
+ynh_backup "/etc/php/7.0/fpm/pool.d/$app.conf"
 
 #=================================================
 # BACKUP THE MYSQL DATABASE

From 789eeae1bd5f40ebeb1891b127dd3fc2766113a6 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 00:28:41 +0200
Subject: [PATCH 16/19] Restore backup before upgrade + cleaning

---
 scripts/upgrade | 23 ++++++++---------------
 1 file changed, 8 insertions(+), 15 deletions(-)

diff --git a/scripts/upgrade b/scripts/upgrade
index 92642dd..2146ddb 100644
--- a/scripts/upgrade
+++ b/scripts/upgrade
@@ -54,14 +54,14 @@ fi
 #=================================================
 ynh_print_info "Backing up the app before upgrading (may take a while)..."
 
-## Backup the current version of the app
-#ynh_backup_before_upgrade
-#ynh_clean_setup () {
-#	# restore it if the upgrade fails
-#	ynh_restore_upgradebackup
-#}
-## Exit if an error occurs during the execution of the script
-#ynh_abort_if_errors
+# Backup the current version of the app
+ynh_backup_before_upgrade
+ynh_clean_setup () {
+	# restore it if the upgrade fails
+	ynh_restore_upgradebackup
+}
+# Exit if an error occurs during the execution of the script
+ynh_abort_if_errors
 
 #=================================================
 # CHECK THE PATH
@@ -107,14 +107,7 @@ ynh_add_fpm_config
 #=================================================
 # SPECIFIC UPGRADE
 #=================================================
-# ...
-#=================================================
-
 ynh_print_info "Running specific upgrade..."
-# Verify the checksum and backup the file if it's different
-#ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE"
-# Recalculate and store the config file checksum into the app settings
-#ynh_store_file_checksum "$final_path/CONFIG_FILE"
 
 # Needed so no manual operation has to be done by user while upgrading
 # If missing, Cheky is stuck in an "upgrade" state and the config file

From 7598c72dafa0926da0ae4142b040f2871d0df1b6 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 00:45:11 +0200
Subject: [PATCH 17/19] update readme

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index 709bd32..8b7c0cc 100644
--- a/README.md
+++ b/README.md
@@ -58,6 +58,7 @@ sudo yunohost app upgrade cheky --url https://github.com/YunoHost-Apps/cheky_ynh
 
 * 4.4.1~ynh4 (11 March 2019)
   * changed: upgrade Cheky to 4.4.1
+  * fix automated upgrade method
 * 4.4~ynh4 (4 March 2019)
   * removed: Debian Jessie support is dropped in favor to Stretch only (Yunohost version > 3)
   * changed: follow lastest rules from <https://github.com/YunoHost/example_ynh>

From 59db15d75085c381ce8033d3ea4da446b7815a03 Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 12:17:18 +0200
Subject: [PATCH 18/19] Update README.md

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index 8b7c0cc..99a4981 100644
--- a/README.md
+++ b/README.md
@@ -59,6 +59,7 @@ sudo yunohost app upgrade cheky --url https://github.com/YunoHost-Apps/cheky_ynh
 * 4.4.1~ynh4 (11 March 2019)
   * changed: upgrade Cheky to 4.4.1
   * fix automated upgrade method
+  * fix scripts `backup` and `restore` - [md-clone](https://github.com/md-clone) with [#19](https://github.com/YunoHost-Apps/cheky_ynh/pull/19)
 * 4.4~ynh4 (4 March 2019)
   * removed: Debian Jessie support is dropped in favor to Stretch only (Yunohost version > 3)
   * changed: follow lastest rules from <https://github.com/YunoHost/example_ynh>

From a635dc813197633a7fea4f3afbbb9cef3a69d84f Mon Sep 17 00:00:00 2001
From: Gofannon <17145502+Gofannon@users.noreply.github.com>
Date: Sat, 11 May 2019 12:26:29 +0200
Subject: [PATCH 19/19] Update README.md

---
 README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 99a4981..fb37048 100644
--- a/README.md
+++ b/README.md
@@ -1,6 +1,6 @@
 # Cheky for YunoHost
 
-[![Integration level](https://dash.yunohost.org/integration/cheky.svg)](https://ci-apps.yunohost.org/jenkins/job/cheky%20%28Community%29/lastBuild/consoleFull)
+[![Integration level](https://dash.yunohost.org/integration/cheky.svg)](https://dash.yunohost.org/appci/app/cheky)
 [![Install Cheky with YunoHost](https://install-app.yunohost.org/install-with-yunohost.png)](https://install-app.yunohost.org/?app=cheky)
 
 > *This package allow you to install cheky quickly and simply on a YunoHost server.