mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-25 15:05:49 +03:00
[GH-ISSUE #613] Integration Disabled #450
Labels
No labels
bug
bug
bug
feature
good-first-issue
new integration
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/healthchecks#450
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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.
@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).
@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:
@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.
@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: