-
Notifications
You must be signed in to change notification settings - Fork 637
Include missing uwsgi.ini #238
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
Conversation
|
SearXNG docker only works properly if every file is owned by root for some reason. If you follow the repo instructions closely and put it inside |
|
I don't put my user in docker group. Is it an expected part of Docker installs? |
Yes. However, I am in the docker user group, and did not get the uwsgi.ini file. |
|
Make sure you have this in your volumes:
- ./searxng:/etc/searxng:rw(https://github.com/searxng/searxng-docker/blob/master/docker-compose.yaml#L55-L56) This will create a If in doubt and you already have that directory, stop your containers, rename that directory and start your containers again. That way it should re-create the directory and put all the files in it. You can then copy the settings from the renamed directory in there and restart the whole show, so you won't lose any settings or the like. EDIT: With this being the relevant log: |
Looking at the README I'd venture it's highly implicit that the folder must be root, as anyone following the README, will get an error when running way forwardEither way, I'd argue some change is needed. E.g. explicitly add the |
|
Or installing it in |
|
This is probably not a good idea because the UWSGI config is updated dynamically based on environment variables (e.g. different amount of workers / threads), which would no longer work then. |
There is probably a reason why this file is in the gitignore, and not included in the source tree, but I'm opening this PR to start discussion on proper fix, as the docker compose doesn't work without it.
Possible fixes:
This fixes #115