mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-24 22:45:56 +03:00
[PR #708] [MERGED] Webhooks support for $BODY placeholder #1011
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#1011
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?
📋 Pull Request Information
Original PR: https://github.com/healthchecks/healthchecks/pull/708
Author: @bdd
Created: 9/24/2022
Status: ✅ Merged
Merged: 9/26/2022
Merged by: @cuu508
Base:
master← Head:body-in-webhooks📝 Commits (1)
2c3c47aWebhooks support for $BODY placeholder📊 Changes
2 files changed (+66 additions, -12 deletions)
View changed files
📝
hc/api/tests/test_notify_webhook.py(+44 -1)📝
hc/api/transports.py(+22 -11)📄 Description
Adds support for sending ping body contents with webhooks.
Only allowed in the 'body' template of the webhook configuration.
Currently only e-mail transport supports including the ping body in the notification.
Implementing this for webhooks seemed like the right place for maximum flexibility.
Example uses:
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.