linuxserver / docker-bookstack

A Docker container for the BookStack documentation wiki
GNU General Public License v3.0
747 stars 108 forks source link

Compose example variables and -e documentation incorrect #150

Closed samwozencroft closed 1 year ago

samwozencroft commented 1 year ago

Is there an existing issue for this?

Current Behavior

DB_USER= for specifying the database user DB_PASS= for specifying the database password

These details are incorrect for the variable for bookstack, It should be DB_USERNAME and DB_PASSWORD

Expected Behavior

No response

Steps To Reproduce

DB_USER= for specifying the database user DB_PASS= for specifying the database password

Environment

- OS: Raspbian GNU/Linux 10 (buster)
- How docker service was installed: CLI

CPU architecture

armhf

Docker creation

docker-compose up

Container logs

bookstack       | /etc/cont-init.d/50-config: line 102: warning: command substitution: ignored null byte in input
bookstack       | /etc/cont-init.d/50-config: line 102: warning: command substitution: ignored null byte in input
bookstack       |
bookstack       |    Illuminate\Database\QueryException
bookstack       |
bookstack       |   SQLSTATE[HY000] [1045] Access denied for user 'admin'@'172.23.0.1' (using password: YES) (SQL: select * from information_schema.tables where table_schema = bookstackapp and table_name = migrations and table_type = 'BASE TABLE')
bookstack       |
bookstack       |   at /app/www/vendor/laravel/framework/src/Illuminate/Database/Connection.php:712
bookstack       |     708▕         // If an exception occurs when attempting to run a query, we'll format the error
bookstack       |     709▕         // message to include the bindings with SQL, which will make this exception a
bookstack       |     710▕         // lot more helpful to the developer instead of just the database's errors.
bookstack       |     711▕         catch (Exception $e) {
bookstack       |   ➜ 712▕             throw new QueryException(
bookstack       |     713▕                 $query, $this->prepareBindings($bindings), $e
bookstack       |     714▕             );
bookstack       |     715▕         }
bookstack       |     716▕     }
bookstack       |
bookstack       |       +33 vendor frames
bookstack       |   34  /app/www/artisan:37
bookstack       |       Illuminate\Foundation\Console\Kernel::handle()
bookstack       | cont-init: info: /etc/cont-init.d/50-config exited 0
github-actions[bot] commented 1 year ago

Thanks for opening your first issue here! Be sure to follow the bug or feature issue templates!

j0nnymoe commented 1 year ago

Dupe https://github.com/linuxserver/docker-bookstack/issues/129