Skip to content

Automatically create integration webhooks when missing [RUN AS-IF-FOSS]

Markus Koller requested to merge 335218-integration-service-hooks into master

What does this MR do?

Previously we were silently failing if the webhook could not be saved when the integration was edited, which led to further exceptions later when the integration was trying to execute the webhook.

The webhook is really only used to associate records in web_hook_logs, so we can create it automatically when needed and don't have to backfill existing integrations.

We now also raise errors if the webhook cannot be saved, which should help us diagnose why this is failing in the first place.

Addresses #335218 (closed), and would make #335685 (closed) redundant.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related to #335218 (closed)

Edited by Markus Koller

Merge request reports

Loading