[PR #737] [MERGED] Allow to retrieve a ping's body #1020

Closed
opened 2026-02-26 00:30:39 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/healthchecks/healthchecks/pull/737
Author: @lablans
Created: 11/23/2022
Status: Merged
Merged: 11/28/2022
Merged by: @cuu508

Base: masterHead: feature/retrievecheckswithbody


📝 Commits (10+)

  • 6790e04 Allow to retrieve pings with their bodies via the API
  • 68cf550 Reformat using black
  • 0dadf30 Add documentation
  • ea84f60 Revert ?with_body=true variant
  • 27951a2 Allow to retrieve pings with their bodies via the API (via extra endpoint)
  • a13879c Add documentation
  • 3a95336 Reformat using black
  • eca71ba Merge branch 'master' into feature/retrievecheckswithbody
  • 4824c89 Remove unnecessary annotation
  • f9cdf9e Construct body_url using django's reverse()

📊 Changes

6 files changed (+140 additions, -8 deletions)

View changed files

📝 hc/api/models.py (+7 -0)
📝 hc/api/tests/test_ping.py (+15 -0)
📝 hc/api/urls.py (+5 -0)
📝 hc/api/views.py (+24 -0)
📝 templates/docs/api.html (+41 -4)
📝 templates/docs/api.md (+48 -4)

📄 Description

This PR adds a new query parameter to the /api/v1/checks/<uuid>/pings/ API endpoint. If set to true, returned pings will contain their respective request bodies (if present). See the added documentation for more information.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/healthchecks/healthchecks/pull/737 **Author:** [@lablans](https://github.com/lablans) **Created:** 11/23/2022 **Status:** ✅ Merged **Merged:** 11/28/2022 **Merged by:** [@cuu508](https://github.com/cuu508) **Base:** `master` ← **Head:** `feature/retrievecheckswithbody` --- ### 📝 Commits (10+) - [`6790e04`](https://github.com/healthchecks/healthchecks/commit/6790e046c9fbcf4494c5fdea2b8312588d31e351) Allow to retrieve pings with their bodies via the API - [`68cf550`](https://github.com/healthchecks/healthchecks/commit/68cf55006d3a496a9bcb16d54b84a206db2ed574) Reformat using black - [`0dadf30`](https://github.com/healthchecks/healthchecks/commit/0dadf3007f86d286d196b24ae841d3ab754e7f93) Add documentation - [`ea84f60`](https://github.com/healthchecks/healthchecks/commit/ea84f60aba93daaaed488855d9599d19a38f1f6f) Revert ?with_body=true variant - [`27951a2`](https://github.com/healthchecks/healthchecks/commit/27951a24a517dd079fecdda94e764de91c681139) Allow to retrieve pings with their bodies via the API (via extra endpoint) - [`a13879c`](https://github.com/healthchecks/healthchecks/commit/a13879c4a279167dae9cf6056ea316cdd8bc6923) Add documentation - [`3a95336`](https://github.com/healthchecks/healthchecks/commit/3a95336a0b74be4234329acdec3ad018020e4bac) Reformat using black - [`eca71ba`](https://github.com/healthchecks/healthchecks/commit/eca71ba523959bfd8ee483dc5913ee196e6703c2) Merge branch 'master' into feature/retrievecheckswithbody - [`4824c89`](https://github.com/healthchecks/healthchecks/commit/4824c89a60bb808ea4ebf31e3a278c5ad021f5c3) Remove unnecessary annotation - [`f9cdf9e`](https://github.com/healthchecks/healthchecks/commit/f9cdf9ec2a716bd86b007824e7bfdbc6aa2ad1fe) Construct body_url using django's reverse() ### 📊 Changes **6 files changed** (+140 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `hc/api/models.py` (+7 -0) 📝 `hc/api/tests/test_ping.py` (+15 -0) 📝 `hc/api/urls.py` (+5 -0) 📝 `hc/api/views.py` (+24 -0) 📝 `templates/docs/api.html` (+41 -4) 📝 `templates/docs/api.md` (+48 -4) </details> ### 📄 Description This PR adds a new query parameter to the `/api/v1/checks/<uuid>/pings/` API endpoint. If set to true, returned pings will contain their respective request bodies (if present). See the added documentation for more information. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 00:30:39 +03:00
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#1020
No description provided.