Skip to content
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

accept_applications in config file doesn't work #18

Open
bumbleblue opened this issue May 16, 2018 · 2 comments
Open

accept_applications in config file doesn't work #18

bumbleblue opened this issue May 16, 2018 · 2 comments
Labels

Comments

@bumbleblue
Copy link
Collaborator

cf 41355f4 and the brute-force solution here: b7d879e

@bumbleblue bumbleblue added the bug label May 16, 2018
@djfun
Copy link
Contributor

djfun commented Sep 29, 2018

looking at config.py, it seems that environment variables have priority over the config files. Do you have the env variable ACCEPT_APPLICATIONS set, which would override the config entry?
Otherwise I cannot reproduce the issue on my machine.

@alexanderkoller
Copy link
Owner

I switched most configuration elements from the config file to the environment variables when we migrated to Docker, because they are easier to configure (through the docker-compose config file or the sloppy.io web interface). It is possible that we forgot some in the heat of the moment.

In my opinion, everything should be configurable either through the config file or env variables (but not both), and where possible, things should be configured through the env variables.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants