[GH-ISSUE #613] Integration Disabled #450

Closed
opened 2026-02-25 23:42:30 +03:00 by kerem · 4 comments
Owner

Originally created by @lettore on GitHub (Mar 1, 2022).
Original GitHub issue: https://github.com/healthchecks/healthchecks/issues/613

Hello,
I'm using the webook integration for Mattermost, I don't know why because I can't see any error, but happens after sometimes that I don't receive any messages and when I go to integrations page I see the mattermost integration is disabled.
If I do test I receive the message, but I don't get any messages from alarms.
The only way I can resolve is to delete and recreate the integration.

Originally created by @lettore on GitHub (Mar 1, 2022). Original GitHub issue: https://github.com/healthchecks/healthchecks/issues/613 Hello, I'm using the webook integration for Mattermost, I don't know why because I can't see any error, but happens after sometimes that I don't receive any messages and when I go to integrations page I see the mattermost integration is disabled. If I do test I receive the message, but I don't get any messages from alarms. The only way I can resolve is to delete and recreate the integration.
kerem closed this issue 2026-02-25 23:42:30 +03:00
Author
Owner

@cuu508 commented on GitHub (Mar 2, 2022):

The integration gets marked as "Disabled" when the webhook returns HTTP 404 – in that case Healthchecks assumes the webhook does not exist anymore (the integration was removed on the receiver side, or the whole channel was removed).

<!-- gh-comment-id:1056363984 --> @cuu508 commented on GitHub (Mar 2, 2022): The integration gets marked as "Disabled" when the webhook returns HTTP 404 – in that case Healthchecks assumes the webhook does not exist anymore (the integration was removed on the receiver side, or the whole channel was removed).
Author
Owner

@lettore commented on GitHub (Mar 2, 2022):

I see, this happens because sometimes the Mattermost server is down. You
can understand that disabling a notification channel without any advice is
not the best way to deal with such problem. It would be possible at least
to re-enable it when a test is successful? The best it would be to have the
choice to decide if enable such behavior, because I think it's not totally
wanted in most scenarios.

Il Mer 2 Mar 2022, 07:33 Pēteris Caune @.***> ha
scritto:

The integration gets marked as "Disabled" when the webhook returns HTTP
404 – in that case Healthchecks assumes the webhook does not exist anymore
(the integration was removed on the receiver side, or the whole channel was
removed).


Reply to this email directly, view it on GitHub
https://github.com/healthchecks/healthchecks/issues/613#issuecomment-1056363984,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJLFEZFJMPJQON5N6BCN7TU54DTRANCNFSM5PVMGJSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID:
@.***>

<!-- gh-comment-id:1056635087 --> @lettore commented on GitHub (Mar 2, 2022): I see, this happens because sometimes the Mattermost server is down. You can understand that disabling a notification channel without any advice is not the best way to deal with such problem. It would be possible at least to re-enable it when a test is successful? The best it would be to have the choice to decide if enable such behavior, because I think it's not totally wanted in most scenarios. Il Mer 2 Mar 2022, 07:33 Pēteris Caune ***@***.***> ha scritto: > The integration gets marked as "Disabled" when the webhook returns HTTP > 404 – in that case Healthchecks assumes the webhook does not exist anymore > (the integration was removed on the receiver side, or the whole channel was > removed). > > — > Reply to this email directly, view it on GitHub > <https://github.com/healthchecks/healthchecks/issues/613#issuecomment-1056363984>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AIJLFEZFJMPJQON5N6BCN7TU54DTRANCNFSM5PVMGJSA> > . > Triage notifications on the go with GitHub Mobile for iOS > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> > or Android > <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. > > You are receiving this because you authored the thread.Message ID: > ***@***.***> >
Author
Owner

@cuu508 commented on GitHub (Mar 9, 2022):

Thanks for the feedback! I've now split apart the Slack and Mattermost notification delivery logic, which was previously shared. For Slack integrations, 404 will still be treated as a permanent failure, and the integration will be marked as disabled. For Mattermost integrations, 404 will now be treated as a transient failure, and the integration will not be marked as disabled.

<!-- gh-comment-id:1062661172 --> @cuu508 commented on GitHub (Mar 9, 2022): Thanks for the feedback! I've now split apart the Slack and Mattermost notification delivery logic, which was previously shared. For Slack integrations, 404 will still be treated as a permanent failure, and the integration will be marked as disabled. For Mattermost integrations, 404 will now be treated as a transient failure, and the integration will *not* be marked as disabled.
Author
Owner

@lettore commented on GitHub (Mar 9, 2022):

Amazing, thank you

Il giorno mer 9 mar 2022 alle ore 09:15 Pēteris Caune <
@.***> ha scritto:

Thanks for the feedback! I've now split apart the Slack and Mattermost
notification delivery logic, which was previously shared. For Slack
integrations, 404 will still be treated as a permanent failure, and the
integration will be marked as disabled. For Mattermost integrations, 404
will now be treated as a transient failure, and the integration will not
be marked as disabled.


Reply to this email directly, view it on GitHub
https://github.com/healthchecks/healthchecks/issues/613#issuecomment-1062661172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIJLFEY3XV2HUFPWIA7BYH3U7BM3PANCNFSM5PVMGJSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID:
@.***>

<!-- gh-comment-id:1062773289 --> @lettore commented on GitHub (Mar 9, 2022): Amazing, thank you Il giorno mer 9 mar 2022 alle ore 09:15 Pēteris Caune < ***@***.***> ha scritto: > Thanks for the feedback! I've now split apart the Slack and Mattermost > notification delivery logic, which was previously shared. For Slack > integrations, 404 will still be treated as a permanent failure, and the > integration will be marked as disabled. For Mattermost integrations, 404 > will now be treated as a transient failure, and the integration will *not* > be marked as disabled. > > — > Reply to this email directly, view it on GitHub > <https://github.com/healthchecks/healthchecks/issues/613#issuecomment-1062661172>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/AIJLFEY3XV2HUFPWIA7BYH3U7BM3PANCNFSM5PVMGJSA> > . > Triage notifications on the go with GitHub Mobile for iOS > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> > or Android > <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>. > > You are receiving this because you authored the thread.Message ID: > ***@***.***> >
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/healthchecks#450
No description provided.