search

gpedro / slack-webhook

Slack WebHook Integration for Java
MIT License
182 stars 54 forks source link

remove slack's URL requirement - allows to use with MatterMost and ot… #20

Closed rpardini closed 8 years ago

rpardini commented 8 years ago

…her Slack-compatible clones

rpardini commented 8 years ago

I've simply removed the check, since MatterMost's urls are unpredictable. My current webhooks urls look like "https://mattermost.my.domain.com/hooks/xxxxtokenxxxx", so maybe a check for "/hooks/" could be implemented, instead of just removing everything.

rpardini commented 8 years ago

Also, BTW, slack-webhook works perfectly with MatterMost, with absolutely no other changes required.