Closed Matthijsy closed 2 years ago
@Matthijsy thanks for the PR! We'll have a look and test it before merging 😉
Please have a look at the linter errors:
pylint elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py
************* Module module
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:85:0: C0325: Unnecessary parens after 'not' keyword (superfluous-parens)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:37:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:44:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:44:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:60:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:66:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:69:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:77:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:77:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:86:0: W1405: Quote delimiter " is inconsistent with the rest of the file (inconsistent-quotes)
elkserver/docker/redelk-base/redelkinstalldata/scripts/modules/slack/module.py:86:12: W1203: Use lazy % formatting in logging functions (logging-fstring-interpolation)
Thank you! Wasn't aware of the linters, but the issues should be resolved now
@Matthijsy Hey I think we're missing the addition to the config.json file. Could you please check? After that it should be good for merging.
@MarcOverIP Sorry for my late response! I think you refer to this file right? https://github.com/outflanknl/RedELK/blob/master/elkserver/mounts/redelk-config/etc/redelk/config.json#L29
For some reason the slack webhook_url config was already in there, but just not used anywhere in the product. I just used that one since the naming sounds logical.
You are right. Thank you for the PR! Merging now.
Currently there is a configuration for notifications via Slack, however there is not slack implementation within the project. This pull requests adds a slack connector, based on the msteams connector. It will send a message to a certain slack webhook when an alarm is raised