-
Notifications
You must be signed in to change notification settings - Fork 37
Add Dockerfile and fix Blacklist #8
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
Fix Blacklist
1 similar comment
1 similar comment
1 similar comment
…, integrate the blacklist into the config
1 similar comment
1 similar comment
|
Awesome! |
|
Please be aware that there were a couple more changes besides the ones mentioned in the initial commit, especially how the blacklist is configured now and how configuration options can be set - this shouldn't break anything for existing users, but some people may be surprised that their blacklist.txt doesn't work anymore ;-) |
|
Roger! /Tilly |
|
I will! We're "launching" it in our office today, so i'll probably hold off for a couple of days, but expect more PRs from me in the future ;-) |
The blacklist was read asynchronously and didn't really stop any blacklisted user from sending commands - now it does.
The Dockerfile creates an Alpine based container that includes the config and blacklist at build time.
ToDo: environment variables to configure the Docker containers