mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2026-07-22 01:17:57 +00:00
Webhooks not enabled for push events cannot be tested using the "Test delivery" button, because the built-in test payload corresponds to a push event and is therefore filtered out at delivery time if the webhook isn't configured to trigger for such events. This fixes it by delivering the payload for a push event regardless of the webhook's configuration. This has the downside of delivering a payload which isn't necessarily representative of what the webhook will deliver for real, but it would be a significant effort to implement test payloads for all possible event types. We leave this as a follow-up improvement. Fixes #7934. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/11073 Reviewed-by: oliverpool <oliverpool@noreply.codeberg.org> Reviewed-by: Gusted <gusted@noreply.codeberg.org> Co-authored-by: Antonin Delpeuch <antonin@delpeuch.eu> Co-committed-by: Antonin Delpeuch <antonin@delpeuch.eu> |
||
|---|---|---|
| .. | ||
| shared | ||
| sourcehut | ||
| TestPushCommits | ||
| default.go | ||
| default_test.go | ||
| deliver.go | ||
| deliver_test.go | ||
| dingtalk.go | ||
| dingtalk_test.go | ||
| discord.go | ||
| discord_test.go | ||
| feishu.go | ||
| feishu_test.go | ||
| general.go | ||
| general_test.go | ||
| gogs.go | ||
| main_test.go | ||
| matrix.go | ||
| matrix_test.go | ||
| msteams.go | ||
| msteams_test.go | ||
| notifier.go | ||
| notifier_test.go | ||
| packagist.go | ||
| packagist_test.go | ||
| slack.go | ||
| slack_test.go | ||
| telegram.go | ||
| telegram_test.go | ||
| webhook.go | ||
| webhook_test.go | ||
| wechatwork.go | ||