YunoHost-Apps / wordpress_ynh

WordPress package for YunoHost
https://wordpress.org/
GNU General Public License v3.0
26 stars 30 forks source link

Please do not modify some PHP fpm values #189

Open tio-trom opened 2 years ago

tio-trom commented 2 years ago

Many Wordpress websites and plugins require bigger values for the php-fpm /etc/php/8.0/fpm/pool.d. Your defaults of 50M and 65M respectively are too low. Mine are

php_admin_value[upload_max_filesize] = 512M
php_admin_value[memory_limit] = 512M
php_admin_value[post_max_size] = 512M

And I have over 10 WP websites with YNH. Every time YNH pushes an update, it changes these values and I wonder why? If these values are manually changed (for good reasons) can you please leave them as they are?

tituspijean commented 2 years ago

To be fixed by re-introducing the config panel, similarly to https://github.com/YunoHost-Apps/nextcloud_ynh/pull/500.

tio-trom commented 2 years ago

To be fixed by re-introducing the config panel, similarly to YunoHost-Apps/nextcloud_ynh#500.

Awesome! Thank you!

iambabs-com commented 1 year ago

Hi, I'm interested in the subject since I got the same situation, but I haven't understood the solution. Could you be more explicit for newbies like me ? ;-) Thanks in advance.

tituspijean commented 1 year ago

For a "newbie like you", the solution would be "wait for it to be fixed". 😓

To be more explicit, the linked PR reinstated some user-tunable settings in config panel. I was hinting that a similar work needed to be done to add the settings mentioned above.

iambabs-com commented 1 year ago

You mean soon a setting will be on yuno admin panel ? OK then for isntance I still manually change the php conf ;-)