[Feature] Add ability to send RESPONSE_BODY in webhook integration about healthchecks HOT 6 CLOSED

Potentially yes, but I'd say only if and when end users specifically request it. The Slack integration includes the last ping body with no opt-out option, and there have been no complaints.

from healthchecks.

I'd like to try and help out and add this in myself as it feels like it should be a fairly quick thing but cant for the life of me find where the values are set. All the vars are fairly common words in the code base like CODE, STATUS, TAGS 😅 . Any pointers to where the current ones are?

from healthchecks.

I think the functionality is there but not surfaced yet?

from healthchecks.

A $BODY placeholder was implemented in #708.

I'm not sure though how well it would work with Mattermost notifications – $BODY contains raw request body, and if you do something like {"body": "$BODY"}, you would likely run into character escaping issues.

from healthchecks.

Currently, the Slack and Mattermost integrations use the same message template: slack_message.json.

The Slack integration already includes last ping body in the notification (implemented in #735), so the template already supports this. The missing part, to enable it for Mattermost, is in the hc.api.transports.Mattermost class, notify method, fetch the body and pass it as an argument when rendering the template. Here's how the Slack integration does it:

And the next step would be to test it and see if the last ping body looks OK in Mattermost UIs.

from healthchecks.

ok cool, might it be worth surfacing a checkbox on the form to say if you want it passed or not? I guess some people would want to send a bunch of logs to the UI but not forced into sending them via the notification

from healthchecks.