sanathp / statusok

Monitor your Website and APIs from your Computer. Get Notified through Slack, E-mail when your server is down or response time is more than expected.
Other
1.62k stars 197 forks source link

Tries to initialise dingding even if no config is provided for it #35

Closed saward closed 4 years ago

saward commented 5 years ago

As per subject. My config only has slack in it, yet it tries to initialise dingding and has an error.

rmorrison61 commented 5 years ago

I had the same issue, it appears that this problem was tied to the Data structure for the Dingding notification service. I just put through a pull request, which resolved the issue for me: https://github.com/sanathp/statusok/pull/41

CC: @sanathp

sanathp commented 4 years ago

Fixed with this commit . Will publish in next release.