-
Notifications
You must be signed in to change notification settings - Fork 38
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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