Remove mysql option on install + Fix php-fpm settings bug

dragondaddy commented 7 months ago

Problem

If we change php-fpm settings of the app in YunoHost admin, the app becomes unusable with a 502 error.
In /etc/php/8.2/fpm/pool.d/hubzilla.conf we find this : listen = /var/run/php/php7.4-fpm-hubzilla.sock

Solution

We add a variable in _common.sh to make sure that 8.2 is used in php-fpm settings (variable used in the config script)
On an already broken install, a simple update with this solution solves the problem

PR Status

[x] Code finished and ready to be reviewed/tested
[x] The fix/enhancement were manually tested (if applicable)

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

dragondaddy commented 7 months ago

!testme

yunohost-bot commented 7 months ago

May the CI gods be with you!

dragondaddy commented 7 months ago

Update to 8.8.7 as requested in https://github.com/YunoHost-Apps/hubzilla_ynh/issues/165

dragondaddy commented 7 months ago

!testme

yunohost-bot commented 7 months ago

:sunflower:

dragondaddy commented 7 months ago

Hi @ericgaspar ! Rather than forcing mysql to postgresql conversion, which seems quite complicated, I'd suggest we just remove the mysql option for new installs, while for already existing installs we keep using the existing mysql db used (I've tested it, it works).

Last but no least, we get the Level 8 back, woohoo!

dragondaddy commented 7 months ago

!testme

yunohost-bot commented 7 months ago

:stuck_out_tongue_winking_eye:

YunoHost-Apps / hubzilla_ynh