nisabek / icinga2-slack-notifications

Icinga2 notification integration with slack
Apache License 2.0
71 stars 36 forks source link

Add colour for host states #17

Closed Hipska closed 6 years ago

Hipska commented 6 years ago

It came to my attention that notification for down host didn't have any colour. So I have added them in the template.

Also worth noting that they didn't even had the blue colour as what you might expect looking at the code. This might need to be figured out later.

Hipska commented 6 years ago

Hi @nisabek, did you also had a look at the comment in my second paragraph?

nisabek commented 6 years ago

I will try to do some tests today to reproduce/fix. If you find any more bugs let me know.

nisabek commented 6 years ago

@Hipska - I figured it out, "blue" is simply not a color supported by slack attachments. I'll replace it with a default HEX representation of grey.

Reference to slack documentation: https://api.slack.com/docs/message-attachments#attachment_structure