[GH-ISSUE #308] When /fail endpoint is called, include request body in email alert #232

Closed
opened 2026-02-25 23:41:42 +03:00 by kerem · 7 comments
Owner

Originally created by @cuu508 on GitHub (Nov 26, 2019).
Original GitHub issue: https://github.com/healthchecks/healthchecks/issues/308

This would allow failing a check with an explicit message, and getting that message in email:

curl -fsS -X POST -d "No space left on device" https://hc-ping.com/uuid/fail
Originally created by @cuu508 on GitHub (Nov 26, 2019). Original GitHub issue: https://github.com/healthchecks/healthchecks/issues/308 This would allow failing a check with an explicit message, and getting that message in email: curl -fsS -X POST -d "No space left on device" https://hc-ping.com/uuid/fail
kerem closed this issue 2026-02-25 23:41:42 +03:00
Author
Owner

@dmhdev commented on GitHub (Mar 26, 2020):

Hello, this would be my first contribution but I have several years of experience in Python, Django and Linux Server Admin. Mind if I work on this issue?

<!-- gh-comment-id:604499428 --> @dmhdev commented on GitHub (Mar 26, 2020): Hello, this would be my first contribution but I have several years of experience in Python, Django and Linux Server Admin. Mind if I work on this issue?
Author
Owner

@cuu508 commented on GitHub (Mar 27, 2020):

@vdaynm sure, sounds good!

I'd like to do a bigger revamp of the email message (#348) but we can do it a step at a time.

<!-- gh-comment-id:604903145 --> @cuu508 commented on GitHub (Mar 27, 2020): @vdaynm sure, sounds good! I'd like to do a bigger revamp of the email message (#348) but we can do it a step at a time.
Author
Owner

@dmhdev commented on GitHub (Apr 1, 2020):

Can you point me to the method or class that is sending the notification email, and should it be sent to all notification channels or only email?

<!-- gh-comment-id:607381918 --> @dmhdev commented on GitHub (Apr 1, 2020): Can you point me to the method or class that is sending the notification email, and should it be sent to all notification channels or only email?
Author
Owner

@cuu508 commented on GitHub (Apr 1, 2020):

Can you point me to the method or class that is sending the notification email

Here: https://github.com/healthchecks/healthchecks/blob/master/hc/api/transports.py#L57

should it be sent to all notification channels or only email?

Do you mean, should the request body be included in all notification templates? – No, each notification type would need to be considered separately. For example there's no way the request body will fit in in a SMS message.

<!-- gh-comment-id:607436130 --> @cuu508 commented on GitHub (Apr 1, 2020): > Can you point me to the method or class that is sending the notification email Here: https://github.com/healthchecks/healthchecks/blob/master/hc/api/transports.py#L57 > should it be sent to all notification channels or only email? Do you mean, should the request body be included in all notification templates? – No, each notification type would need to be considered separately. For example there's no way the request body will fit in in a SMS message.
Author
Owner

@dlecan commented on GitHub (Jun 25, 2020):

I would like to implement the same feature, but for Slack notifications. Would you like to track this with another issue?

<!-- gh-comment-id:649692024 --> @dlecan commented on GitHub (Jun 25, 2020): I would like to implement the same feature, but for Slack notifications. Would you like to track this with another issue?
Author
Owner

@lukastribus commented on GitHub (Oct 25, 2020):

I'm looking for this feature as well (in email, telegram, pushover integrations).

Seeing the request body without logging into to the dashboard would be extremely convenient, for obvious reasons.

<!-- gh-comment-id:716191392 --> @lukastribus commented on GitHub (Oct 25, 2020): I'm looking for this feature as well (in email, telegram, pushover integrations). Seeing the request body without logging into to the dashboard would be extremely convenient, for obvious reasons.
Author
Owner

@cuu508 commented on GitHub (Dec 26, 2020):

I've updated the email template to include the last ping body:

image

@dlecan, @lukastribus:

The ping body can be 10KB – it may need to be truncated to, say, 1KB for IM notifications. Let's track each in a separate issue. I'm happy to review PRs.

<!-- gh-comment-id:751374257 --> @cuu508 commented on GitHub (Dec 26, 2020): I've updated the email template to include the last ping body: ![image](https://user-images.githubusercontent.com/661859/103155470-0e4e7500-47a9-11eb-91ad-9e1272b25b66.png) @dlecan, @lukastribus: The ping body can be 10KB – it may need to be truncated to, say, 1KB for IM notifications. Let's track each in a separate issue. I'm happy to review PRs.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/healthchecks#232
No description provided.