From 78c1776113f39a2955095da0760960fda96decdf Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:28:40 +0200 Subject: [PATCH 01/34] backup script cleanup --- scripts/backup | 28 +--------------------------- 1 file changed, 1 insertion(+), 27 deletions(-) diff --git a/scripts/backup b/scripts/backup index 8871560e..cdaf227c 100644 --- a/scripts/backup +++ b/scripts/backup @@ -14,10 +14,6 @@ source /usr/share/yunohost/helpers # MANAGE SCRIPT FAILURE #================================================= -ynh_clean_setup () { - ### Remove this function if there's nothing to clean before calling the remove script. - true -} # Exit if an error occurs during the execution of the script ynh_abort_if_errors @@ -56,34 +52,12 @@ ynh_print_info "Backing up php-fpm configuration..." ynh_backup "/etc/php/7.2/fpm/pool.d/$app.conf" #================================================= -# BACKUP THE PostgreSQL DATABASE +# BACKUP THE POSTGRESQL DATABASE #================================================= ynh_print_info "Backing up the PostgreSQL database..." ynh_psql_dump_db "$db_name" > db.sql -#================================================= -# SPECIFIC BACKUP -#================================================= -# BACKUP LOGROTATE -#================================================= -ynh_print_info "Backing up logrotate configuration..." - -ynh_backup "/etc/logrotate.d/$app" - -#================================================= -# BACKUP SYSTEMD -#================================================= -ynh_print_info "Backing up systemd configuration..." - -ynh_backup "/etc/systemd/system/$app.service" - -#================================================= -# BACKUP A CRON FILE -#================================================= - -ynh_backup "/etc/cron.d/$app" - #================================================= # END OF SCRIPT #================================================= From 734cf8778f074d4f44b0487a3e4a5aed99e71ab9 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:30:19 +0200 Subject: [PATCH 02/34] restore script cleanup --- scripts/restore | 32 +------------------------------- 1 file changed, 1 insertion(+), 31 deletions(-) diff --git a/scripts/restore b/scripts/restore index d9035164..9696c169 100644 --- a/scripts/restore +++ b/scripts/restore @@ -14,10 +14,6 @@ source /usr/share/yunohost/helpers # MANAGE SCRIPT FAILURE #================================================= -ynh_clean_setup () { - #### Remove this function if there's nothing to clean before calling the remove script. - true -} # Exit if an error occurs during the execution of the script ynh_abort_if_errors @@ -91,7 +87,7 @@ ynh_print_info "Reinstalling dependencies..." ynh_install_app_dependencies "$pkg_dependencies" #================================================= -# RESTORE THE PostgreSQL DATABASE +# RESTORE THE POSTGRESQL DATABASE #================================================= ynh_print_info "Restoring the PostgreSQL database..." @@ -99,32 +95,6 @@ ynh_psql_test_if_first_run ynh_psql_create_db "$db_name" "$db_name" "$db_pwd" ynh_psql_execute_file_as_root ./db.sql "$db_name" -#================================================= -# RESTORE SYSTEMD -#================================================= -ynh_print_info "Restoring the systemd configuration..." - -ynh_restore_file "/etc/systemd/system/$app.service" -systemctl enable "$app".service - -#================================================= -# ADVERTISE SERVICE IN ADMIN PANEL -#================================================= - -yunohost service add "$app" --log "/var/log/$app/$app.log" - -#================================================= -# RESTORE THE CRON FILE -#================================================= - -ynh_restore_file "/etc/cron.d/$app" - -#================================================= -# RESTORE THE LOGROTATE CONFIGURATION -#================================================= - -ynh_restore_file "/etc/logrotate.d/$app" - #================================================= # GENERIC FINALIZATION #================================================= From e8f8164a3ca05545495bd5b9e651b265281ceffc Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:31:36 +0200 Subject: [PATCH 03/34] Fix final_path owner --- scripts/restore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/restore b/scripts/restore index 9696c169..99761488 100644 --- a/scripts/restore +++ b/scripts/restore @@ -68,7 +68,7 @@ ynh_system_user_create "$app" #================================================= # Restore permissions on app files -chown -R root: "$final_path" +chown -R "$app": "$final_path" #================================================= # RESTORE THE PHP-FPM CONFIGURATION From fe44c98eaac2eab429daafaf86143211adae27e3 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:32:15 +0200 Subject: [PATCH 04/34] Add PHP installation --- scripts/restore | 2 ++ 1 file changed, 2 insertions(+) diff --git a/scripts/restore b/scripts/restore index 99761488..38b43353 100644 --- a/scripts/restore +++ b/scripts/restore @@ -84,6 +84,8 @@ ynh_restore_file "/etc/php/7.2/fpm/pool.d/$app.conf" ynh_print_info "Reinstalling dependencies..." # Define and install dependencies +ynh_install_php --phpversion="7.2" + ynh_install_app_dependencies "$pkg_dependencies" #================================================= From 3b95413fa0fbef84ea82ff23f8a060e54e75c391 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:33:18 +0200 Subject: [PATCH 05/34] move php+fpm restoration after php installation --- scripts/restore | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/scripts/restore b/scripts/restore index 38b43353..549039d4 100644 --- a/scripts/restore +++ b/scripts/restore @@ -70,12 +70,6 @@ ynh_system_user_create "$app" # Restore permissions on app files chown -R "$app": "$final_path" -#================================================= -# RESTORE THE PHP-FPM CONFIGURATION -#================================================= - -ynh_restore_file "/etc/php/7.2/fpm/pool.d/$app.conf" - #================================================= # SPECIFIC RESTORATION #================================================= @@ -97,6 +91,12 @@ ynh_psql_test_if_first_run ynh_psql_create_db "$db_name" "$db_name" "$db_pwd" ynh_psql_execute_file_as_root ./db.sql "$db_name" +#================================================= +# RESTORE THE PHP-FPM CONFIGURATION +#================================================= + +ynh_restore_file "/etc/php/7.2/fpm/pool.d/$app.conf" + #================================================= # GENERIC FINALIZATION #================================================= From 03ed439e003498d84a7419824c09c3b2dfc57b89 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 21:48:23 +0200 Subject: [PATCH 06/34] Activate backup_restore check --- check_process | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/check_process b/check_process index 901667fe..8d6c7569 100644 --- a/check_process +++ b/check_process @@ -10,7 +10,7 @@ setup_private=0 setup_public=0 upgrade=0 - backup_restore=0 + backup_restore=1 multi_instance=0 incorrect_path=0 port_already_use=0 From a735d5718c374d29cfe87c8a0190883e885b6a36 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 22:51:17 +0200 Subject: [PATCH 07/34] Cleanup script --- scripts/upgrade | 18 ------------------ 1 file changed, 18 deletions(-) diff --git a/scripts/upgrade b/scripts/upgrade index 843069f0..87a042b5 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -123,28 +123,10 @@ ynh_add_fpm_config # ... #================================================= -### Verify the checksum of a file, stored by `ynh_store_file_checksum` in the install script. -### And create a backup of this file if the checksum is different. So the file will be backed up if the admin had modified it. ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE" # Recalculate and store the checksum of the file for the next upgrade. ynh_store_file_checksum "$final_path/CONFIG_FILE" -#================================================= -# SETUP LOGROTATE -#================================================= -ynh_print_info "Upgrading logrotate configuration..." - -# Use logrotate to manage app-specific logfile(s) -ynh_use_logrotate --non-append - -#================================================= -# SETUP SYSTEMD -#================================================= -ynh_print_info "Upgrading systemd configuration..." - -# Create a dedicated systemd config -ynh_add_systemd_config - #================================================= # GENERIC FINALIZATION #================================================= From 6a36abe2a6bedc5b6dafe02bec1d1fb537df8209 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 23:09:51 +0200 Subject: [PATCH 08/34] Fix upgarde script --- scripts/upgrade | 36 ++++++++++++++++++++++++++++++++---- 1 file changed, 32 insertions(+), 4 deletions(-) diff --git a/scripts/upgrade b/scripts/upgrade index 87a042b5..8e9922ea 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -99,6 +99,8 @@ ynh_add_nginx_config #================================================= ynh_print_info "Upgrading dependencies..." +ynh_install_php --phpversion="7.2" + ynh_install_app_dependencies "$pkg_dependencies" #================================================= @@ -115,17 +117,43 @@ ynh_system_user_create "$app" ynh_print_info "Upgrading php-fpm configuration..." # Create a dedicated php-fpm config -ynh_add_fpm_config +ynh_add_fpm_config --phpversion="7.2" + +#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released +mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +systemctl reload php7.0-fpm +systemctl reload php7.2-fpm #================================================= # SPECIFIC UPGRADE #================================================= -# ... +# INSTALL PHP DEPENDENCIES #================================================= +chown -R "$app": "$final_path" + +ynh_install_composer --workdir="$final_path" + +#================================================= +# DEPLOYMENT +#================================================= + +pushd "$final_path" + php7.2 artisan config:clear + php7.2 artisan config:cache + php7.2 artisan route:clear + php7.2 artisan route:cache + php7.2 artisan storage:link + php7.2 artisan migrate --force + php7.2 artisan update + php7.2 artisan horizon:purge +popd + + + ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE" # Recalculate and store the checksum of the file for the next upgrade. -ynh_store_file_checksum "$final_path/CONFIG_FILE" +ynh_store_file_checksum "$final_path/.env" #================================================= # GENERIC FINALIZATION @@ -134,7 +162,7 @@ ynh_store_file_checksum "$final_path/CONFIG_FILE" #================================================= # Set permissions on app files -chown -R root: "$final_path" +chown -R "$app": "$final_path" #================================================= # SETUP SSOWAT From 4073e76c1712b01f6555ce454a68f61bebaed5f9 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Sun, 31 Mar 2019 23:15:34 +0200 Subject: [PATCH 09/34] Activate upgrade check --- check_process | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/check_process b/check_process index 8d6c7569..74293a75 100644 --- a/check_process +++ b/check_process @@ -9,7 +9,8 @@ setup_nourl=0 setup_private=0 setup_public=0 - upgrade=0 + upgrade=1 + upgrade=1 from_commit=a2549aa86776cf7a88c0be9c34c4de4cf348f8d3 backup_restore=1 multi_instance=0 incorrect_path=0 From e68231e80dbdcb51d596edf2adf49ebe22e74bbe Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 00:47:40 +0200 Subject: [PATCH 10/34] fix php-fpm upgrade --- scripts/upgrade | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/scripts/upgrade b/scripts/upgrade index 8e9922ea..43e89c46 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -116,6 +116,10 @@ ynh_system_user_create "$app" #================================================= ynh_print_info "Upgrading php-fpm configuration..." +mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +systemctl reload php7.0-fpm +systemctl reload php7.2-fpm + # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" From 2b661d3e2e1875f5b53f2be999440e4c6f2ca44c Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 01:16:05 +0200 Subject: [PATCH 11/34] remove upgrade from non working version --- check_process | 1 - 1 file changed, 1 deletion(-) diff --git a/check_process b/check_process index 74293a75..45e5c5e6 100644 --- a/check_process +++ b/check_process @@ -10,7 +10,6 @@ setup_private=0 setup_public=0 upgrade=1 - upgrade=1 from_commit=a2549aa86776cf7a88c0be9c34c4de4cf348f8d3 backup_restore=1 multi_instance=0 incorrect_path=0 From ec26cf58e131843eb2325c7a571b53de4413a471 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 02:54:54 +0200 Subject: [PATCH 12/34] fix php-fpm --- scripts/install | 5 ++--- scripts/remove | 9 ++++----- scripts/upgrade | 13 +++---------- 3 files changed, 9 insertions(+), 18 deletions(-) diff --git a/scripts/install b/scripts/install index 4c263969..ef9b9e32 100644 --- a/scripts/install +++ b/scripts/install @@ -106,9 +106,8 @@ ynh_print_info "Configuring php-fpm..." # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" -systemctl reload php7.0-fpm +#Workaround waiting 'ynh_add_fpm_config --phpversion='' released +cp "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" systemctl reload php7.2-fpm #================================================= diff --git a/scripts/remove b/scripts/remove index 5784d04e..7bea232a 100644 --- a/scripts/remove +++ b/scripts/remove @@ -47,14 +47,13 @@ ynh_remove_nginx_config #================================================= ynh_print_info "Removing php-fpm configuration" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" -systemctl reload php7.2-fpm -systemctl reload php7.0-fpm - # Remove the dedicated php-fpm config ynh_remove_fpm_config +#Workaround waiting 'ynh_add_fpm_config --phpversion='' released +ynh_secure_remove "/etc/php/7.2/fpm/pool.d/$app.conf" +systemctl reload php7.2-fpm + #================================================= # REMOVE DEPENDENCIES #================================================= diff --git a/scripts/upgrade b/scripts/upgrade index 43e89c46..a8299a8f 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -116,16 +116,11 @@ ynh_system_user_create "$app" #================================================= ynh_print_info "Upgrading php-fpm configuration..." -mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" -systemctl reload php7.0-fpm -systemctl reload php7.2-fpm - # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" -systemctl reload php7.0-fpm +#Workaround waiting 'ynh_add_fpm_config --phpversion='' released +cp -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" systemctl reload php7.2-fpm #================================================= @@ -153,9 +148,7 @@ pushd "$final_path" php7.2 artisan horizon:purge popd - - -ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE" +ynh_backup_if_checksum_is_different "$final_path/.env" # Recalculate and store the checksum of the file for the next upgrade. ynh_store_file_checksum "$final_path/.env" From bb89459f9148e1ada7a188bd61af3beb2c3d7bb5 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 06:12:43 +0200 Subject: [PATCH 13/34] Revert "fix php-fpm" This reverts commit ec26cf58e131843eb2325c7a571b53de4413a471. --- scripts/install | 5 +++-- scripts/remove | 9 +++++---- scripts/upgrade | 13 ++++++++++--- 3 files changed, 18 insertions(+), 9 deletions(-) diff --git a/scripts/install b/scripts/install index ef9b9e32..4c263969 100644 --- a/scripts/install +++ b/scripts/install @@ -106,8 +106,9 @@ ynh_print_info "Configuring php-fpm..." # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Workaround waiting 'ynh_add_fpm_config --phpversion='' released -cp "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released +mv "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +systemctl reload php7.0-fpm systemctl reload php7.2-fpm #================================================= diff --git a/scripts/remove b/scripts/remove index 7bea232a..5784d04e 100644 --- a/scripts/remove +++ b/scripts/remove @@ -47,13 +47,14 @@ ynh_remove_nginx_config #================================================= ynh_print_info "Removing php-fpm configuration" +#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released +mv "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +systemctl reload php7.2-fpm +systemctl reload php7.0-fpm + # Remove the dedicated php-fpm config ynh_remove_fpm_config -#Workaround waiting 'ynh_add_fpm_config --phpversion='' released -ynh_secure_remove "/etc/php/7.2/fpm/pool.d/$app.conf" -systemctl reload php7.2-fpm - #================================================= # REMOVE DEPENDENCIES #================================================= diff --git a/scripts/upgrade b/scripts/upgrade index a8299a8f..43e89c46 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -116,11 +116,16 @@ ynh_system_user_create "$app" #================================================= ynh_print_info "Upgrading php-fpm configuration..." +mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +systemctl reload php7.0-fpm +systemctl reload php7.2-fpm + # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Workaround waiting 'ynh_add_fpm_config --phpversion='' released -cp -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released +mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +systemctl reload php7.0-fpm systemctl reload php7.2-fpm #================================================= @@ -148,7 +153,9 @@ pushd "$final_path" php7.2 artisan horizon:purge popd -ynh_backup_if_checksum_is_different "$final_path/.env" + + +ynh_backup_if_checksum_is_different "$final_path/CONFIG_FILE" # Recalculate and store the checksum of the file for the next upgrade. ynh_store_file_checksum "$final_path/.env" From 7b3a3e78927701a055876c5ae4cfbc11d2861786 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 06:17:25 +0200 Subject: [PATCH 14/34] Fix php-fpm --- scripts/upgrade | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/upgrade b/scripts/upgrade index 43e89c46..4e1d2624 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -117,8 +117,8 @@ ynh_system_user_create "$app" ynh_print_info "Upgrading php-fpm configuration..." mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" -systemctl reload php7.0-fpm systemctl reload php7.2-fpm +systemctl reload php7.0-fpm # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" From 8f7e97629b685ea24757d937874fb85c4c2bc5b5 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 07:30:55 +0200 Subject: [PATCH 15/34] Update php-fpm upgrade --- scripts/upgrade | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/scripts/upgrade b/scripts/upgrade index 4e1d2624..8883b7f8 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -116,7 +116,8 @@ ynh_system_user_create "$app" #================================================= ynh_print_info "Upgrading php-fpm configuration..." -mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released +mv "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" systemctl reload php7.2-fpm systemctl reload php7.0-fpm @@ -124,7 +125,7 @@ systemctl reload php7.0-fpm ynh_add_fpm_config --phpversion="7.2" #Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +mv "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" systemctl reload php7.0-fpm systemctl reload php7.2-fpm From cbe4796a5629514a61757def9ae71852c2af30bd Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 18:18:37 +0200 Subject: [PATCH 16/34] Update version number --- manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifest.json b/manifest.json index 7bec3d1c..2dfbb7d8 100644 --- a/manifest.json +++ b/manifest.json @@ -5,7 +5,7 @@ "description": { "en": "ActivityPub Federated Image Sharing" }, - "version": "0.8.4~ynh1", + "version": "0.8.4~ynh2", "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { From 924d805b4cf96c6c34b485c3dbe5ebf8d4707611 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 19:53:50 +0200 Subject: [PATCH 17/34] Fix php-fpm --- scripts/remove | 8 ++++++++ scripts/upgrade | 6 ++++-- 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/scripts/remove b/scripts/remove index 5784d04e..f8bd8478 100644 --- a/scripts/remove +++ b/scripts/remove @@ -82,6 +82,14 @@ ynh_print_info "Removing the dedicated system user" # Delete a system user ynh_system_user_delete "$app" +#================================================= +# START PHP-FPM 7.0 FPM +#================================================= +ynh_print_info "Start php7.0-fpm" + +#Sometimes with package_check php7.0-fpm fail to reload and stop. So starting the service, just in case. +systemctl start php7.0-fpm + #================================================= # END OF SCRIPT #================================================= diff --git a/scripts/upgrade b/scripts/upgrade index 8883b7f8..ba7bb706 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -117,7 +117,8 @@ ynh_system_user_create "$app" ynh_print_info "Upgrading php-fpm configuration..." #Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" +sleep 5 systemctl reload php7.2-fpm systemctl reload php7.0-fpm @@ -125,7 +126,8 @@ systemctl reload php7.0-fpm ynh_add_fpm_config --phpversion="7.2" #Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" +sleep 5 systemctl reload php7.0-fpm systemctl reload php7.2-fpm From 23b328182ca8f0c6d9fd0614fae6b1a7408ee65e Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 19:55:18 +0200 Subject: [PATCH 18/34] Change version number --- manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifest.json b/manifest.json index 7bec3d1c..e49f9740 100644 --- a/manifest.json +++ b/manifest.json @@ -5,7 +5,7 @@ "description": { "en": "ActivityPub Federated Image Sharing" }, - "version": "0.8.4~ynh1", + "version": "0.8.4~ynh3", "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { From 945fc891d768eab246eb516e7263f01f51f95db3 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 21:19:20 +0200 Subject: [PATCH 19/34] Fix change_url --- scripts/change_url | 48 ++++++++++++++++++++++++++-------------------- 1 file changed, 27 insertions(+), 21 deletions(-) diff --git a/scripts/change_url b/scripts/change_url index 60846630..a9530b5c 100644 --- a/scripts/change_url +++ b/scripts/change_url @@ -14,10 +14,10 @@ source /usr/share/yunohost/helpers #================================================= old_domain=$YNH_APP_OLD_DOMAIN -old_path=$YNH_APP_OLD_PATH +old_path="/" new_domain=$YNH_APP_NEW_DOMAIN -new_path=$YNH_APP_NEW_PATH +new_path="/" app=$YNH_APP_INSTANCE_NAME @@ -43,12 +43,6 @@ then change_domain=1 fi -change_path=0 -if [ "$old_path" != "$new_path" ] -then - change_path=1 -fi - #================================================= # STANDARD MODIFICATIONS #================================================= @@ -58,18 +52,6 @@ ynh_print_info "Updating nginx web server configuration..." nginx_conf_path=/etc/nginx/conf.d/$old_domain.d/$app.conf -# Change the path in the nginx config file -if [ $change_path -eq 1 ] -then - # Make a backup of the original nginx config file if modified - ynh_backup_if_checksum_is_different "$nginx_conf_path" - # Set global variables for nginx helper - domain="$old_domain" - path_url="$new_path" - # Create a dedicated nginx config - ynh_add_nginx_config -fi - # Change the domain for nginx if [ $change_domain -eq 1 ] then @@ -83,9 +65,33 @@ fi #================================================= # SPECIFIC MODIFICATIONS #================================================= -# ... +# UPDATE CONFIG FILE #================================================= +config="$final_path/.env" +ynh_replace_string "APP_URL=.*" "APP_URL=https://$new_domain" "$config" +ynh_replace_string "ADMIN_DOMAIN=.*" "ADMIN_DOMAIN=\"$new_domain\"" "$config" +ynh_replace_string "APP_DOMAIN=.*" "APP_DOMAIN=\"$new_domain\"" "$config" +ynh_replace_string "MAIL_FROM_ADDRESS=.*" "MAIL_FROM_ADDRESS=\"pixelfed@$new_domain\"" "$config" + +#================================================= +# STORE THE CHECKSUM OF THE CONFIG FILE +#================================================= + +# Calculate and store the config file checksum into the app settings +ynh_store_file_checksum "$config" + +#================================================= +# APPLY CHANGES +#================================================= + +pushd "$final_path" + php7.2 artisan config:clear + php7.2 artisan config:cache + php7.2 artisan route:clear + php7.2 artisan route:cache + php7.2 artisan horizon:purge +popd #================================================= # GENERIC FINALISATION #================================================= From 96c861040c0e2a1cfbcb52b44e5017666452a427 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Mon, 1 Apr 2019 21:52:18 +0200 Subject: [PATCH 20/34] Activate additional features --- check_process | 8 ++++---- manifest.json | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/check_process b/check_process index 45e5c5e6..f753394d 100644 --- a/check_process +++ b/check_process @@ -7,12 +7,12 @@ setup_sub_dir=0 setup_root=1 setup_nourl=0 - setup_private=0 - setup_public=0 + setup_private=1 + setup_public=1 upgrade=1 backup_restore=1 - multi_instance=0 - incorrect_path=0 + multi_instance=1 + incorrect_path=1 port_already_use=0 change_url=0 ;;; Levels diff --git a/manifest.json b/manifest.json index e49f9740..7731889c 100644 --- a/manifest.json +++ b/manifest.json @@ -13,9 +13,9 @@ "email": "jean-baptiste@holcroft.fr" }, "requirements": { - "yunohost": ">= 3.0.0" + "yunohost": ">= 3.4" }, - "multi_instance": false, + "multi_instance": true, "services": [ "nginx" ], From 85d148fd9a49eea7a5586eb245ff6604f787653d Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 00:34:59 +0200 Subject: [PATCH 21/34] update helpers --- scripts/_common.sh | 228 +++++++++++++++++++++++++++------------------ 1 file changed, 139 insertions(+), 89 deletions(-) diff --git a/scripts/_common.sh b/scripts/_common.sh index 8e0bc493..05326162 100644 --- a/scripts/_common.sh +++ b/scripts/_common.sh @@ -78,67 +78,15 @@ ynh_install_php () { # Store php_version into the config of this app ynh_app_setting_set $app php_version $phpversion - # Install an extra repo to get multiple php versions - ynh_install_extra_repo --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --name=php - - if [ "$phpversion" == "7.0" ]; then + if [ "$phpversion" == "7.0" ] + then ynh_die "Do not use ynh_install_php to install php7.0" - - # Php 7.1 - elif [ "$phpversion" == "7.1" ]; then - # Get the current version available for libpcre3 on packages.sury.org - local libpcre3_version=$(apt-cache madison "libpcre3" | grep "packages.sury.org" | tail -n1 | awk '{print $3}') - - # equivs doesn't handle correctly this dependence. - # Force the upgrade of libpcre3 for php7.1 - ynh_package_install "libpcre3=$libpcre3_version" - - local php_dependencies="php7.1, php7.1-fpm" - - # Php 7.2 - elif [ "$phpversion" == "7.2" ]; then - # Get the current version available for libpcre3 on packages.sury.org - local libpcre3_version=$(apt-cache madison "libpcre3" | grep "packages.sury.org" | tail -n1 | awk '{print $3}') - - # equivs doesn't handle correctly this dependence. - # Force the upgrade of libpcre3 for php7.2 - ynh_package_install "libpcre3=$libpcre3_version" - - local php_dependencies="php7.2, php7.2-fpm" - - # Php 7.3 - elif [ "$phpversion" == "7.3" ]; then - # Get the current version available for libpcre2-8-0 on packages.sury.org - local libpcre2_version=$(apt-cache madison "libpcre2-8-0" | grep "packages.sury.org" | tail -n1 | awk '{print $3}') - - # equivs doesn't handle correctly this dependence. - # Force the upgrade of libpcre2-8-0 for php7.3 - ynh_package_install "libpcre2-8-0=$libpcre2_version" - - local php_dependencies="php7.3, php7.3-fpm" - - else - ynh_die "The version $phpversion of php isn't handle by this helper." fi # Store the ID of this app and the version of php requested for it - echo "$YNH_APP_ID:$phpversion" | tee --append "/etc/php/ynh_app_version" + echo "$YNH_APP_INSTANCE_NAME:$phpversion" | tee --append "/etc/php/ynh_app_version" - # Build a control file for equivs-build - echo "Section: misc -Priority: optional -Package: php${phpversion}-ynh-deps -Version: 1.0 -Depends: $php_dependencies -Architecture: all -Description: Fake package for php_$phpversion dependencies - This meta-package is only responsible of installing its dependencies." \ - > /tmp/php_${phpversion}-ynh-deps.control - - # Install the fake package for php - ynh_package_install_from_equivs /tmp/php_${phpversion}-ynh-deps.control \ - || ynh_die --message="Unable to install dependencies" - ynh_secure_remove /tmp/php_${phpversion}-ynh-deps.control + ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="php$phpversion php${phpversion}-fpm php${phpversion}-common" # Advertise service in admin panel yunohost service add php${phpversion}-fpm --log "/var/log/php${phpversion}-fpm.log" @@ -158,37 +106,13 @@ ynh_remove_php () { fi # Remove the line for this app - sed --in-place "/$YNH_APP_ID:$phpversion/d" "/etc/php/ynh_app_version" + sed --in-place "/$YNH_APP_INSTANCE_NAME:$phpversion/d" "/etc/php/ynh_app_version" # If no other app uses this version of php, remove it. if ! grep --quiet "$phpversion" "/etc/php/ynh_app_version" then - # Remove the metapackage for php - ynh_package_autopurge php${phpversion}-ynh-deps - # Then remove php-fpm php-cli for this version. - # The previous command won't remove them, but we have to remove those package to clean php - ynh_package_autopurge php${phpversion}-fpm php${phpversion}-cli - - if [ "$phpversion" == "7.1" ] || [ "$phpversion" == "7.2" ] - then - # Do not restore libpcre3 if php7.1 or 7.2 is still used. - if ! grep --quiet --extended-regexp "7.1|7.2" "/etc/php/ynh_app_version" - then - # Get the current version available for libpcre3 on the standard repo - local libpcre3_version=$(apt-cache madison "libpcre3" | grep "debian.org" | tail -n1 | awk '{print $3}') - - # Force to reinstall the standard version of libpcre3 - ynh_package_install --allow-downgrades libpcre3=$libpcre3_version >&2 - fi - elif [ "$phpversion" == "7.3" ] - then - # Get the current version available for libpcre2-8-0 on the standard repo - local libpcre2_version=$(apt-cache madison "libpcre2-8-0" | grep "debian.org" | tail -n1 | awk '{print $3}') - - # Force to reinstall the standard version of libpcre2-8-0 - ynh_package_install --allow-downgrades libpcre2-8-0=$libpcre2_version - fi - + # Purge php dependences for this version. + ynh_package_autopurge "php$phpversion php${phpversion}-fpm php${phpversion}-common" # Remove the service from the admin panel yunohost service remove php${phpversion}-fpm fi @@ -196,7 +120,6 @@ ynh_remove_php () { # If no other app uses alternate php versions, remove the extra repo for php if [ ! -s "/etc/php/ynh_app_version" ] then - ynh_remove_extra_repo --name=php ynh_secure_remove /etc/php/ynh_app_version fi } @@ -208,7 +131,7 @@ ynh_remove_php () { #================================================= # Pin a repository. # -# usage: ynh_pin_repo --package=packages --pin=pin_filter --priority=priority_value [--name=name] [--append] +# usage: ynh_pin_repo --package=packages --pin=pin_filter [--priority=priority_value] [--name=name] [--append] # | arg: -p, --package - Packages concerned by the pin. Or all, *. # | arg: -i, --pin - Filter for the pin. # | arg: -p, --priority - Priority for the pin @@ -289,17 +212,19 @@ ynh_add_repo () { # Add an extra repository correctly, pin it and get the key. # -# usage: ynh_install_extra_repo --repo="repo" [--key=key_url] [--name=name] [--append] +# usage: ynh_install_extra_repo --repo="repo" [--key=key_url] [--priority=priority_value] [--name=name] [--append] # | arg: -r, --repo - Complete url of the extra repository. # | arg: -k, --key - url to get the public key. +# | arg: -p, --priority - Priority for the pin # | arg: -n, --name - Name for the files for this repo, $app as default value. # | arg: -a, --append - Do not overwrite existing files. ynh_install_extra_repo () { # Declare an array to define the options of this helper. - local legacy_args=rkna - declare -Ar args_array=( [r]=repo= [k]=key= [n]=name= [a]=append ) + local legacy_args=rkpna + declare -Ar args_array=( [r]=repo= [k]=key= [p]=priority= [n]=name= [a]=append ) local repo local key + local priority local name local append # Manage arguments with getopts @@ -307,6 +232,7 @@ ynh_install_extra_repo () { name="${name:-$app}" append=${append:-0} key=${key:-0} + priority=${priority:-} if [ $append -eq 1 ] then @@ -337,7 +263,12 @@ ynh_install_extra_repo () { # Build $pin from the uri without http and any sub path local pin="${uri#*://}" pin="${pin%%/*}" - ynh_pin_repo --package="*" --pin="origin \"$pin\"" --name="$name" $append + # Set a priority only if asked + if [ -n "$priority" ] + then + priority="--priority=$priority" + fi + ynh_pin_repo --package="*" --pin="origin \"$pin\"" $priority --name="$name" $append # Get the public key for the repo if [ -n "$key" ] @@ -377,3 +308,122 @@ ynh_remove_extra_repo () { # Update the list of package to exclude the old repo ynh_package_update } + +# Install packages from an extra repository properly. +# +# usage: ynh_install_extra_app_dependencies --repo="repo" --package="dep1 dep2" [--key=key_url] [--name=name] +# | arg: -r, --repo - Complete url of the extra repository. +# | arg: -p, --package - The packages to install from this extra repository +# | arg: -k, --key - url to get the public key. +# | arg: -n, --name - Name for the files for this repo, $app as default value. +ynh_install_extra_app_dependencies () { + # Declare an array to define the options of this helper. + local legacy_args=rpkn + declare -Ar args_array=( [r]=repo= [p]=package= [k]=key= [n]=name= ) + local repo + local package + local key + local name + # Manage arguments with getopts + ynh_handle_getopts_args "$@" + name="${name:-$app}" + key=${key:-0} + + # Set a key only if asked + if [ -n "$key" ] + then + key="--key=$key" + fi + # Add an extra repository for those packages + ynh_install_extra_repo --repo="$repo" $key --priority=995 --name=$name + + # Install requested dependencies from this extra repository. + ynh_add_app_dependencies --package="$package" + + # Remove this extra repository after packages are installed + ynh_remove_extra_repo --name=$app +} + +#================================================= + +# patched version of ynh_install_app_dependencies to be used with ynh_add_app_dependencies + +# Define and install dependencies with a equivs control file +# This helper can/should only be called once per app +# +# usage: ynh_install_app_dependencies dep [dep [...]] +# | arg: dep - the package name to install in dependence +# You can give a choice between some package with this syntax : "dep1|dep2" +# Example : ynh_install_app_dependencies dep1 dep2 "dep3|dep4|dep5" +# This mean in the dependence tree : dep1 & dep2 & (dep3 | dep4 | dep5) +# +# Requires YunoHost version 2.6.4 or higher. +ynh_install_app_dependencies () { + local dependencies=$@ + dependencies="$(echo "$dependencies" | sed 's/\([^\<=\>]\)\ \([^(]\)/\1, \2/g')" + dependencies=${dependencies//|/ | } + local 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 + + local version=$(grep '\"version\": ' "$manifest_path" | cut -d '"' -f 4) # Retrieve the version number in the manifest file. + if [ ${#version} -eq 0 ]; then + version="1.0" + fi + local dep_app=${app//_/-} # Replace all '_' by '-' + + # Handle specific versions + if [[ "$dependencies" =~ [\<=\>] ]] + then + # Replace version specifications by relationships syntax + # https://www.debian.org/doc/debian-policy/ch-relationships.html + # Sed clarification + # [^(\<=\>] ignore if it begins by ( or < = >. To not apply twice. + # [\<=\>] matches < = or > + # \+ matches one or more occurence of the previous characters, for >= or >>. + # [^,]\+ matches all characters except ',' + # Ex: package>=1.0 will be replaced by package (>= 1.0) + dependencies="$(echo "$dependencies" | sed 's/\([^(\<=\>]\)\([\<=\>]\+\)\([^,]\+\)/\1 (\2 \3)/g')" + fi + + cat > /tmp/${dep_app}-ynh-deps.control << EOF # Make a control file for equivs-build +Section: misc +Priority: optional +Package: ${dep_app}-ynh-deps +Version: ${version} +Depends: ${dependencies} +Architecture: all +Description: Fake package for $app (YunoHost app) dependencies + This meta-package is only responsible of installing its dependencies. +EOF + ynh_package_install_from_equivs /tmp/${dep_app}-ynh-deps.control \ + || ynh_die --message="Unable to install dependencies" # Install the fake package and its dependencies + rm /tmp/${dep_app}-ynh-deps.control + ynh_app_setting_set --app=$app --key=apt_dependencies --value="$dependencies" +} + +ynh_add_app_dependencies () { + # Declare an array to define the options of this helper. + local legacy_args=pr + declare -Ar args_array=( [p]=package= [r]=replace) + local package + local replace + # Manage arguments with getopts + ynh_handle_getopts_args "$@" + replace=${replace:-0} + + local current_dependencies="" + if [ $replace -eq 0 ] + then + local dep_app=${app//_/-} # Replace all '_' by '-' + if ynh_package_is_installed --package="${dep_app}-ynh-deps" + then + current_dependencies="$(dpkg-query --show --showformat='${Depends}' ${dep_app}-ynh-deps) " + fi + + current_dependencies=${current_dependencies// | /|} + fi + + ynh_install_app_dependencies "${current_dependencies}${package}" +} From df0d304d275cfb010f96bf222dcf852b352efec7 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 00:55:10 +0200 Subject: [PATCH 22/34] fix dearmor --- scripts/_common.sh | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/scripts/_common.sh b/scripts/_common.sh index 05326162..f6a9538d 100644 --- a/scripts/_common.sh +++ b/scripts/_common.sh @@ -274,13 +274,7 @@ ynh_install_extra_repo () { if [ -n "$key" ] then mkdir -p "/etc/apt/trusted.gpg.d" - if [[ "$(basename "$key")" =~ ".asc" ]] - then - local key_ext=asc - else - local key_ext=gpg - fi - wget -q "$key" -O - | gpg --dearmor | $wget_append /etc/apt/trusted.gpg.d/$name.$key_ext > /dev/null + wget -q "$key" -O - | gpg --dearmor | $wget_append /etc/apt/trusted.gpg.d/$name.gpg > /dev/null fi # Update the list of package with the new repo From c1070c3eeff62530a9683f8e90b8edccd8e5d046 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 03:04:01 +0200 Subject: [PATCH 23/34] upgrade helper --- scripts/_common.sh | 14 ++++++++++---- scripts/install | 14 +++++++++++++- scripts/restore | 21 ++++++++++++++++----- scripts/upgrade | 14 +++++++++++++- 4 files changed, 52 insertions(+), 11 deletions(-) diff --git a/scripts/_common.sh b/scripts/_common.sh index f6a9538d..f036f1df 100644 --- a/scripts/_common.sh +++ b/scripts/_common.sh @@ -5,12 +5,18 @@ #================================================= # dependencies used by the app -pkg_dependencies="php7.2-pgsql php7.2-mbstring php7.2-bcmath php7.2-simplexml php7.2-curl postgresql redis-server \ -php7.2-intl php7.2-exif \ -libfreetype6 libjpeg62-turbo libpng16-16 libxpm4 libvpx4 libwebp6 libmagickwand-6.q16-3 \ -php7.2-gd \ +pkg_dependencies="postgresql redis-server \ +libfreetype6 libjpeg62-turbo libpng16-16 libxpm4 libvpx4 libwebp6 libmagickwand-6.q16-3\ pngquant jpegoptim gifsicle" +extra_pkg_dependencies="php7.2-pgsql php7.2-mbstring php7.2-bcmath php7.2-simplexml php7.2-curl php7.2-intl php7.2-exif php7.2-gd" +# +# +# +# +# +# +# #================================================= # PERSONAL HELPERS #================================================= diff --git a/scripts/install b/scripts/install index 4c263969..8849cdab 100644 --- a/scripts/install +++ b/scripts/install @@ -52,9 +52,21 @@ ynh_app_setting_set "$app" is_public "$is_public" #================================================= ynh_print_info "Installing dependencies..." +ynh_install_app_dependencies "$pkg_dependencies" + +#================================================= +# INSTALL PHP +#================================================= +ynh_print_info "Installing php..." + ynh_install_php --phpversion="7.2" -ynh_install_app_dependencies "$pkg_dependencies" +#================================================= +# INSTALL PHP DEPENDENCIES +#================================================= +ynh_print_info "Installing php dependencies..." + +ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" #================================================= # CREATE A POSTGRESQL DATABASE diff --git a/scripts/restore b/scripts/restore index 549039d4..6b36ebf4 100644 --- a/scripts/restore +++ b/scripts/restore @@ -73,15 +73,26 @@ chown -R "$app": "$final_path" #================================================= # SPECIFIC RESTORATION #================================================= -# REINSTALL DEPENDENCIES +# INSTALL DEPENDENCIES #================================================= -ynh_print_info "Reinstalling dependencies..." - -# Define and install dependencies -ynh_install_php --phpversion="7.2" +ynh_print_info "Installing dependencies..." ynh_install_app_dependencies "$pkg_dependencies" +#================================================= +# INSTALL PHP +#================================================= +ynh_print_info "Installing php..." + +ynh_install_php --phpversion="7.2" + +#================================================= +# INSTALL PHP DEPENDENCIES +#================================================= +ynh_print_info "Installing php dependencies..." + +ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" + #================================================= # RESTORE THE POSTGRESQL DATABASE #================================================= diff --git a/scripts/upgrade b/scripts/upgrade index ba7bb706..e39f6a90 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -99,9 +99,21 @@ ynh_add_nginx_config #================================================= ynh_print_info "Upgrading dependencies..." +ynh_install_app_dependencies "$pkg_dependencies" + +#================================================= +# UPGRADE PHP +#================================================= +ynh_print_info "Upgrading php..." + ynh_install_php --phpversion="7.2" -ynh_install_app_dependencies "$pkg_dependencies" +#================================================= +# UPGRADE PHP DEPENDENCIES +#================================================= +ynh_print_info "Upgrading php dependencies..." + +ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" #================================================= # CREATE DEDICATED USER From d4f59e6baabe2e624f46a37e7a11af1461111d37 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 04:00:07 +0200 Subject: [PATCH 24/34] Fix dependencies --- scripts/_common.sh | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/scripts/_common.sh b/scripts/_common.sh index f036f1df..b1ccd2a6 100644 --- a/scripts/_common.sh +++ b/scripts/_common.sh @@ -6,17 +6,11 @@ # dependencies used by the app pkg_dependencies="postgresql redis-server \ -libfreetype6 libjpeg62-turbo libpng16-16 libxpm4 libvpx4 libwebp6 libmagickwand-6.q16-3\ +libfreetype6 libjpeg62-turbo libpng16-16 libxpm4 libvpx4 libmagickwand-6.q16-3 libwebp6 \ pngquant jpegoptim gifsicle" -extra_pkg_dependencies="php7.2-pgsql php7.2-mbstring php7.2-bcmath php7.2-simplexml php7.2-curl php7.2-intl php7.2-exif php7.2-gd" -# -# -# -# -# -# -# +extra_pkg_dependencies="php7.2-bcmath php7.2-cli php7.2-curl php7.2-exif php7.2-gd php7.2-intl php7.2-json php7.2-mbstring php7.2-pgsql php7.2-simplexml php7.2-xml php7.2-zip" + #================================================= # PERSONAL HELPERS #================================================= From 0942da60bf49798a65d7061263e8c98270d2ab3a Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 04:01:42 +0200 Subject: [PATCH 25/34] Update manifest.json --- manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifest.json b/manifest.json index 7731889c..e1337fe9 100644 --- a/manifest.json +++ b/manifest.json @@ -5,7 +5,7 @@ "description": { "en": "ActivityPub Federated Image Sharing" }, - "version": "0.8.4~ynh3", + "version": "0.8.4~ynh5", "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { From bcb6775b6aea926c6db6ed5658a5a9e60ba79273 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Tue, 2 Apr 2019 14:39:52 +0200 Subject: [PATCH 26/34] upgrade ynh_install_php helper --- scripts/_common.sh | 23 ++++++++++++++++------- scripts/install | 9 +-------- scripts/restore | 9 +-------- scripts/upgrade | 9 +-------- 4 files changed, 19 insertions(+), 31 deletions(-) diff --git a/scripts/_common.sh b/scripts/_common.sh index b1ccd2a6..926b5524 100644 --- a/scripts/_common.sh +++ b/scripts/_common.sh @@ -62,18 +62,20 @@ ynh_install_composer () { || ynh_die "Unable to update core dependencies with Composer." } - # Install another version of php. # -# usage: ynh_install_php --phpversion=phpversion +# usage: ynh_install_php --phpversion=phpversion [--package=packages] # | arg: -v, --phpversion - Version of php to install. Can be one of 7.1, 7.2 or 7.3 +# | arg: -p, --package - Additionnal php packages to install ynh_install_php () { # Declare an array to define the options of this helper. - local legacy_args=v - declare -Ar args_array=( [v]=phpversion= ) + local legacy_args=vp + declare -Ar args_array=( [v]=phpversion= [p]=package= ) local phpversion + local package # Manage arguments with getopts ynh_handle_getopts_args "$@" + package=${package:-} # Store php_version into the config of this app ynh_app_setting_set $app php_version $phpversion @@ -86,7 +88,16 @@ ynh_install_php () { # Store the ID of this app and the version of php requested for it echo "$YNH_APP_INSTANCE_NAME:$phpversion" | tee --append "/etc/php/ynh_app_version" - ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="php$phpversion php${phpversion}-fpm php${phpversion}-common" + # Add an extra repository for those packages + ynh_install_extra_repo --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --priority=995 --name=extra_php_version + + # Install requested dependencies from this extra repository. + # Install php-fpm first, otherwise php will install apache as a dependency. + ynh_add_app_dependencies --package="php${phpversion}-fpm" + ynh_add_app_dependencies --package="php$phpversion php${phpversion}-common $package" + + # Remove this extra repository after packages are installed + ynh_remove_extra_repo --name=extra_php_version # Advertise service in admin panel yunohost service add php${phpversion}-fpm --log "/var/log/php${phpversion}-fpm.log" @@ -124,8 +135,6 @@ ynh_remove_php () { fi } - - #================================================= # FUTURE OFFICIAL HELPERS #================================================= diff --git a/scripts/install b/scripts/install index 8849cdab..18b18b72 100644 --- a/scripts/install +++ b/scripts/install @@ -59,14 +59,7 @@ ynh_install_app_dependencies "$pkg_dependencies" #================================================= ynh_print_info "Installing php..." -ynh_install_php --phpversion="7.2" - -#================================================= -# INSTALL PHP DEPENDENCIES -#================================================= -ynh_print_info "Installing php dependencies..." - -ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" +ynh_install_php --phpversion="7.2" --package="$extra_pkg_dependencies" #================================================= # CREATE A POSTGRESQL DATABASE diff --git a/scripts/restore b/scripts/restore index 6b36ebf4..6d5ccc72 100644 --- a/scripts/restore +++ b/scripts/restore @@ -84,14 +84,7 @@ ynh_install_app_dependencies "$pkg_dependencies" #================================================= ynh_print_info "Installing php..." -ynh_install_php --phpversion="7.2" - -#================================================= -# INSTALL PHP DEPENDENCIES -#================================================= -ynh_print_info "Installing php dependencies..." - -ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" +ynh_install_php --phpversion="7.2" --package="$extra_pkg_dependencies" #================================================= # RESTORE THE POSTGRESQL DATABASE diff --git a/scripts/upgrade b/scripts/upgrade index e39f6a90..f3561f26 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -106,14 +106,7 @@ ynh_install_app_dependencies "$pkg_dependencies" #================================================= ynh_print_info "Upgrading php..." -ynh_install_php --phpversion="7.2" - -#================================================= -# UPGRADE PHP DEPENDENCIES -#================================================= -ynh_print_info "Upgrading php dependencies..." - -ynh_install_extra_app_dependencies --repo="https://packages.sury.org/php/ $(lsb_release -sc) main" --key="https://packages.sury.org/php/apt.gpg" --package="$extra_pkg_dependencies" +ynh_install_php --phpversion="7.2" --package="$extra_pkg_dependencies" #================================================= # CREATE DEDICATED USER From 784657c4a05762a8afc45f5f4b985740f8c74c50 Mon Sep 17 00:00:00 2001 From: Jean-Baptiste Holcroft Date: Thu, 4 Apr 2019 11:33:20 +0200 Subject: [PATCH 27/34] Fix upgrade after restoration --- scripts/restore | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/restore b/scripts/restore index 549039d4..0e73f36a 100644 --- a/scripts/restore +++ b/scripts/restore @@ -88,7 +88,7 @@ ynh_install_app_dependencies "$pkg_dependencies" ynh_print_info "Restoring the PostgreSQL database..." ynh_psql_test_if_first_run -ynh_psql_create_db "$db_name" "$db_name" "$db_pwd" +ynh_psql_setup_db "$db_name" "$db_name" "$db_pwd" ynh_psql_execute_file_as_root ./db.sql "$db_name" #================================================= From cd60251b5df58a90f56a28ef8820ecbb25728ce3 Mon Sep 17 00:00:00 2001 From: Jean-Baptiste Holcroft Date: Thu, 4 Apr 2019 11:53:11 +0200 Subject: [PATCH 28/34] fix path traversal --- conf/nginx.conf | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/conf/nginx.conf b/conf/nginx.conf index 67b55d0a..14cbc899 100644 --- a/conf/nginx.conf +++ b/conf/nginx.conf @@ -1,4 +1,4 @@ -location __PATH__ { +location __PATH__/ { # Path to source alias __FINALPATH__/public/ ; From 18ec744550d697be745901be289f281e7efa7aba Mon Sep 17 00:00:00 2001 From: lapineige Date: Tue, 9 Apr 2019 17:52:54 +0200 Subject: [PATCH 29/34] Add registration config information Some people might want to change the registration to false, for instance after creating their (single user instance) account. --- README.md | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/README.md b/README.md index 137a97f5..b71d6e9c 100644 --- a/README.md +++ b/README.md @@ -42,6 +42,11 @@ After being first registered, you need to execute the folloing command to promot and respond yes to the question ` Add admin privileges to this user?` +### Allow/Close registration + +Registrations are open by default. +To change that setting, edit `/var/www/pixelfed/.env` and set `OPEN_REGISTRATION=false` instead of `true`. + ## Documentation * [Official documentation](https://docs.pixelfed.org/master/) From 0754e9d668446004f40ed2a124df996d38674e5b Mon Sep 17 00:00:00 2001 From: yalh76 Date: Wed, 10 Apr 2019 02:10:56 +0200 Subject: [PATCH 30/34] Update to 0.8.6 --- README.md | 2 +- conf/.env | 9 ++++----- conf/app.src | 4 ++-- manifest.json | 2 +- 4 files changed, 8 insertions(+), 9 deletions(-) diff --git a/README.md b/README.md index 137a97f5..9a01d31b 100644 --- a/README.md +++ b/README.md @@ -9,7 +9,7 @@ If you don't have YunoHost, please see [here](https://yunohost.org/#/install) to ## Overview The federated image sharing service Pixelfed, for YunoHost -**Shipped version:** 0.8.4 +**Shipped version:** 0.8.6 I'm waiting for an official release (alpha/beta or stable) to invest more time on this package. Don't hesitate to give a hand if you wish, I assume only the nginx file needs improvments. diff --git a/conf/.env b/conf/.env index ccd91fe0..53d5334f 100644 --- a/conf/.env +++ b/conf/.env @@ -44,11 +44,6 @@ OPEN_REGISTRATION=true RECAPTCHA_ENABLED=false ENFORCE_EMAIL_VERIFICATION=true -PUSHER_APP_ID= -PUSHER_APP_KEY= -PUSHER_APP_SECRET= -PUSHER_APP_CLUSTER=mt1 - MAX_PHOTO_SIZE=15000 MAX_CAPTION_LENGTH=150 MAX_ALBUM_LENGTH=4 @@ -70,3 +65,7 @@ HORIZON_DARKMODE=true # php artisan optimize ACTIVITY_PUB=false REMOTE_FOLLOW=false + +CS_BLOCKED_DOMAINS='example.org,example.net,example.com' +CS_CW_DOMAINS='example.org,example.net,example.com' +CS_UNLISTED_DOMAINS='example.org,example.net,example.com' \ No newline at end of file diff --git a/conf/app.src b/conf/app.src index 30825423..a62243bc 100644 --- a/conf/app.src +++ b/conf/app.src @@ -1,5 +1,5 @@ -SOURCE_URL=https://github.com/pixelfed/pixelfed/archive/ac53aea28884f0c49c110ac6d87a7b40e92171c9.tar.gz -SOURCE_SUM=ca6156026eb1bb8d8995d247a895b88078e93f1127b75af07058d74f57830846 +SOURCE_URL=https://github.com/pixelfed/pixelfed/archive/v0.8.6.tar.gz +SOURCE_SUM=56acbfe18752693d3312207d6955e65c9f6aa73c7e801a224248b2d4b8e2884f OURCE_SUM_PRG=sha256sum SOURCE_FORMAT=tar.gz SOURCE_IN_SUBDIR=true \ No newline at end of file diff --git a/manifest.json b/manifest.json index 7731889c..99aed292 100644 --- a/manifest.json +++ b/manifest.json @@ -5,7 +5,7 @@ "description": { "en": "ActivityPub Federated Image Sharing" }, - "version": "0.8.4~ynh3", + "version": "0.8.6~ynh1", "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { From 77171f736474205c11bf455a37ef266d144a8d4e Mon Sep 17 00:00:00 2001 From: yalh76 Date: Wed, 10 Apr 2019 02:35:33 +0200 Subject: [PATCH 31/34] deactivate DEBUG --- conf/.env | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/conf/.env b/conf/.env index ccd91fe0..966ee9e5 100644 --- a/conf/.env +++ b/conf/.env @@ -1,7 +1,7 @@ APP_NAME=__APP__ APP_ENV=production APP_KEY= -APP_DEBUG=true +APP_DEBUG=false APP_URL=https://__DOMAIN__ ADMIN_DOMAIN="__DOMAIN__" From f2e5a39f3592b6437e087e635553cbfa95b30449 Mon Sep 17 00:00:00 2001 From: yalh76 Date: Wed, 10 Apr 2019 11:04:23 +0200 Subject: [PATCH 32/34] Activate federation --- conf/.env | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/conf/.env b/conf/.env index ccd91fe0..bbf8080a 100644 --- a/conf/.env +++ b/conf/.env @@ -68,5 +68,5 @@ HORIZON_DARKMODE=true # php artisan cache:clear # php artisan optimize:clear # php artisan optimize -ACTIVITY_PUB=false -REMOTE_FOLLOW=false +ACTIVITY_PUB=true +REMOTE_FOLLOW=true From 14530f541c1bc9714d4d7e9249c22df39730d96f Mon Sep 17 00:00:00 2001 From: yalh76 Date: Wed, 10 Apr 2019 14:23:37 +0200 Subject: [PATCH 33/34] Fix spelling --- manifest.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifest.json b/manifest.json index 7731889c..bd82f4b9 100644 --- a/manifest.json +++ b/manifest.json @@ -9,7 +9,7 @@ "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { - "name": "jibec", + "name": "Jean-Baptiste Holcroft", "email": "jean-baptiste@holcroft.fr" }, "requirements": { From f29dc5e884098eb4f50b197ac05e440998dd47ed Mon Sep 17 00:00:00 2001 From: yalh76 Date: Wed, 10 Apr 2019 22:16:15 +0200 Subject: [PATCH 34/34] Remove ugly move needed before yunohost 3.5.2 --- manifest.json | 4 ++-- scripts/install | 5 ----- scripts/remove | 5 ----- scripts/upgrade | 12 ------------ 4 files changed, 2 insertions(+), 24 deletions(-) diff --git a/manifest.json b/manifest.json index 7731889c..1f08e492 100644 --- a/manifest.json +++ b/manifest.json @@ -5,7 +5,7 @@ "description": { "en": "ActivityPub Federated Image Sharing" }, - "version": "0.8.4~ynh3", + "version": "0.8.4~ynh4", "url": "https://pixelfed.org/", "license": "AGPL-3.0-or-later", "maintainer": { @@ -13,7 +13,7 @@ "email": "jean-baptiste@holcroft.fr" }, "requirements": { - "yunohost": ">= 3.4" + "yunohost": ">= 3.5" }, "multi_instance": true, "services": [ diff --git a/scripts/install b/scripts/install index 4c263969..9ef0f9b0 100644 --- a/scripts/install +++ b/scripts/install @@ -106,11 +106,6 @@ ynh_print_info "Configuring php-fpm..." # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" -systemctl reload php7.0-fpm -systemctl reload php7.2-fpm - #================================================= # INSTALL PHP DEPENDENCIES #================================================= diff --git a/scripts/remove b/scripts/remove index f8bd8478..0489390e 100644 --- a/scripts/remove +++ b/scripts/remove @@ -47,11 +47,6 @@ ynh_remove_nginx_config #================================================= ynh_print_info "Removing php-fpm configuration" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" -systemctl reload php7.2-fpm -systemctl reload php7.0-fpm - # Remove the dedicated php-fpm config ynh_remove_fpm_config diff --git a/scripts/upgrade b/scripts/upgrade index ba7bb706..aa0017bb 100644 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -116,21 +116,9 @@ ynh_system_user_create "$app" #================================================= ynh_print_info "Upgrading php-fpm configuration..." -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv -f "/etc/php/7.2/fpm/pool.d/$app.conf" "/etc/php/7.0/fpm/pool.d/$app.conf" -sleep 5 -systemctl reload php7.2-fpm -systemctl reload php7.0-fpm - # Create a dedicated php-fpm config ynh_add_fpm_config --phpversion="7.2" -#Ugly move waiting 'ynh_add_fpm_config --phpversion='' released -mv -f "/etc/php/7.0/fpm/pool.d/$app.conf" "/etc/php/7.2/fpm/pool.d/$app.conf" -sleep 5 -systemctl reload php7.0-fpm -systemctl reload php7.2-fpm - #================================================= # SPECIFIC UPGRADE #=================================================