YunoHost-Apps / bookstack_ynh

BookStack package for YunoHost

https://www.bookstackapp.com/

GNU General Public License v3.0

12 stars 3 forks source link

After update, logs not working, images gone. #36

Closed tr1p0p closed 2 years ago

tr1p0p commented 3 years ago

So, after update on RPI 4 with yunohost install. EVERY update Bookstack it's break ALL images path, and also break access for the admin account.

Basically every update is pretty scary, i don't even know how it can lost ALL images at every update, can't make a wiki with this..

What can i do ?

tr1p0p commented 3 years ago

Screenshot 2021-07-19 at 14-08-46 NullWiki

ericgaspar commented 3 years ago

Sorry for the problem. As I am not using myself the app and no one as reported so far it seems that the bug went under the radar... Not sure what the problem is. I created a branch where PHP as been updated to version 8.0. If you are willing to test you feed back will be appreciated sudo yunohost app upgrade bookstack -u https://github.com/YunoHost-Apps/bookstack_ynh/tree/patch --force

tr1p0p commented 3 years ago

So, update with this seem to run fine, but still a "unknown error occurred" when trying to log in..

marcelcosta commented 2 years ago

I have this error too!

If you need any information, please ask.

da-am commented 2 years ago

I've been getting this for a few months as well. The folder permissions look ok, bookstack www-data. Is there something to look for to fix this?

weex commented 2 years ago

Does https://www.bookstackapp.com/docs/admin/debugging/#broken-links-or-no-images-after-app_url-change help?

I didn't see an issue about this in upstream but feel free to search and create an issue to get help from those more knowledgable about this software. https://github.com/BookStackApp/BookStack/issues/

da-am commented 2 years ago

Thanks for that! My logs showed database errors left and right. I was going to start over but saw that there is a migrate command for the database so I ran php artisan migrate. Now everything works again. I guess the last update didn't fire off the migrate command properly?

weex commented 2 years ago

Interesting, the script looks ok to me but yes if you have any logs around the update they may be helpful for future troubleshooting. Feel free to close this if your issue is resolved.

marcelcosta commented 2 years ago

I did the job for me too!

Thanks.

ericgaspar commented 2 years ago

38 should solve the issue of lost images after upgrade.

Bewar, we still have issues with the change url script not working in all configuration.