search

YunoHost-Apps / my_webapp_ynh

Custom Web app with SFTP access
GNU General Public License v3.0
48 stars 39 forks source link

YoloMerge provoked a permission error #50

Closed Jaxom99 closed 3 years ago

Jaxom99 commented 4 years ago

Hi there,

I installed a php app (InvoicePlane) inside a webapp some time ago. Recently, stuff broke (no pdf generation available via php) and displayed message of wrong permissions. After some debug and looking into php config files of Yunohost, it looks like the user/group was changed from webapp1 to my_webapp in ths commit (https://github.com/YunoHost-Apps/my_webapp_ynh/commit/66cc5be9f8a41090ad38d71032ec7ec6ec366330) (with a really obvious commit message :stuck_out_tongue_closed_eyes: ) but the config in my host did not change.

Doing a manual chmod in the /var/www/my_webapp folder fixed it !

So, maybe this issue will help some folk around here :smile_cat:

alexAubin commented 4 years ago

Indeed what happens in the upgrade script wasnt consistent with the install script, should be fixed now