-
Notifications
You must be signed in to change notification settings - Fork 40
Fixing docker example command for latest alpine image #44
base: master
Are you sure you want to change the base?
Conversation
Fixing docker example command for alpine image
I just got stumped by this. Would be nice to have the PR merged to update the docs. |
That is not correct. The default config file for docker points to the /filebrowser.db |
The default config points to filebrowser.db but the docker command maps a file on the local filesystem called filebrowser.db to one on the docker container's file system called database.db To fix this we can either update the default config to expect a file called "database.db" or do what this PR suggested and update the docker command so that a file called "filebrowser.db" appears on the container file system For what its worth, my install on docker did not work out of the box and these fixes were required. |
I'm not sure I'm following you. This is the default database path https://github.com/filebrowser/filebrowser/blob/master/docker_config.json#L6C30-L6C30 |
I think the confusion is arising because we are referencing two different files.
I think you would probably have a more in-depth knowledge than me about which file is actually being used |
You are referrencing a linuxserver based image config, but actually changing a regular docker file. It uses docker_config.json which points to the /database.db |
My apologies, I understand now. Thanks for the clarification. |
While checking out filebrowser, I just noticed the database file is in the latest version of the alpine image in the same location as for the non-alpine image.
/database/filebrowser.db
This simply adjusts the example docker run command of the install docs