seyren icon indicating copy to clipboard operation
seyren copied to clipboard

Published 20 hours ago • verified publisher icon scobal

Use Slack webhook instead of API token

Open vichheann opened this issue 8 years ago • 6 comments

Should fix #376. We may change the message using slack attachment though.

vichheann avatar Mar 29 '16 19:03 vichheann

I'll test this and let you know

alenkacz avatar Mar 30 '16 18:03 alenkacz

This might be quite a widely used integration so it might be worth considering making this backwards compatible

scobal avatar May 01 '16 19:05 scobal

Hi, sorry for my late reply, I was very busy. I have reverted my changes to keep the slack integration with the API token for backward compatibility and re-added the webhook support. Hope this is better now !

vichheann avatar May 28 '16 22:05 vichheann

any update on merging this? also +1 on using message attachments. that would make the alerts in slack much more readable.

garrettux avatar Aug 30 '16 20:08 garrettux

Personally, I'd rather see SLACK_WEBHOOK_URL override SLACK_TOKEN and not the other way around.

jcampos8782 avatar Dec 15 '16 07:12 jcampos8782

I built on this branch and opened up #423

jcampos8782 avatar Dec 16 '16 08:12 jcampos8782