Closed Bug 1767084 Opened 3 years ago Closed 3 years ago

[WEBHOOKS] Disable webhooks after large number of errors and notify webhook owner.

Categories

(bugzilla.mozilla.org :: Extensions, enhancement)

enhancement

Tracking

()

RESOLVED FIXED

People

(Reporter: dkl, Assigned: dkl)

References

Details

Attachments

(1 file)

  • After a specified number of failures, the webhook will be automatically disabled and no future calls will be made.
  • Bugzilla will send a notification to the webhook owner when the failure threshold has been hit and the webhook is disabled. The owner can then try to figure out what is wrong and re-enable it.

Thanks for the CC on this; what is the number of failures that will disable the webhook? Or is that a setting/field on setup of the webhooks themselves?

(In reply to Jira Integration Bot from comment #1)

Thanks for the CC on this; what is the number of failures that will disable the webhook? Or is that a setting/field on setup of the webhooks themselves?

The disabling code has not been added yet but it will definitely have a value that is configurable. At first it would just be a global value but it is possible we could extend it to be per webhook. We could even have a group of webhook owners where the cut off does not apply. Either way there will be a notification sent out when a hook is disabled.

Where will/what type of the notification be sent? (Will it be an API to the endpoint associated with the service?)

Attached file GitHub Pull Request

(In reply to Bryan Sieber [ :bsieber ] from comment #3)

Where will/what type of the notification be sent? (Will it be an API to the endpoint associated with the service?)

Initially it will be just an email to the webhook author, containing the webhook name, the last error, and a link to the webhooks preference page. We can work out ways to optimize this later if you have other ideas that will help your usage.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: