solidnerd / docker-bookstack

BookStack in a container
MIT License
466 stars 196 forks source link

Installation instructions outdated #512

Open kekcsi opened 1 month ago

kekcsi commented 1 month ago

Hello, I was trying to build and install a bookstack using your image. I faced a number of issues. First, the version uploaded to docker hub is rather old, with bookstack v22.04. I could install but I needed the newer features, so I came to build one myself. Second, the "docker run" commands in the README.md don't work with the newer version of bookstack. They lack the -e APP_KEY=whatever\ line. Even if I add that, the app can't start properly, instead of login, I get an "unexpected error" page. Furthermore, you cannot mount the .env file because the startup scripts would want to rename it and fail. So the modified docker run command also doesn't work as suggested in the README.md file. As opposed to the compose yaml, where there is mysql 8.3, we still get the old 5.7.21 version of mysql in README.md "docker run" commands. With docker compose, the server will start up and show a login page, but the default admin@admin.com user name is not accepted because the new bookstack version demands a real e-mail address. Anyway thanks for your hard work, I hope this helps to improve the documentation. Workaround is to compose and then enter: docker exec -u root -it docker-bookstack-master_bookstack_1 bash root@60***a61:/var/www/bookstack# php artisan bookstack:create-admin But I didn't yet find the correct way to do the installation steps mentioned.

ssddanbrown commented 1 month ago

First, the version uploaded to docker hub is rather old, with bookstack v22.04. I could install but I needed the newer features, so I came to build one myself.

New versions are uploaded to docker hub as can be seen here, and latest points the the current release, and the commands in the readme & compose refer to the latest version. It's just that docker hub shows an old version by default for some reason.

but the default admin@admin.com user name is not accepted because the new bookstack version demands a real e-mail address.

This should not be the case, at least that's not something that's changed in BookStack.