Configurable json body size for healthchecks hooks (don't truncate) #294

New Issue

nightah · 2020-02-18T07:47:18Z

nightah commented

2020-02-18 07:47:18 +00:00

What I'm trying to do and why

When using HTTP POST to the Healthchecks endpoint, you can include arbitrary payload in the request body. If the request body looks like a UTF-8 string, Healthchecks.io will log the first 10 kilobytes of the request body, so you can inspect it later.

This limit is now configurable/removable as part of 58a118c494.

Due to this I would like if we can configure the current truncation so that the full contents of a log can be POSTed to Healthchecks instead of the default 10kb.

Other notes / implementation ideas

This is a continuation of #241 and #249, to allow the full body of the borgmatic output to get to Healthchecks.

#### What I'm trying to do and why When using HTTP POST to the Healthchecks endpoint, you can include arbitrary payload in the request body. If the request body looks like a UTF-8 string, Healthchecks.io will log the first 10 kilobytes of the request body, so you can inspect it later. This limit is now configurable/removable as part of https://github.com/healthchecks/healthchecks/commit/58a118c49499ed814d3aedf3a761bc83624b460d. Due to this I would like if we can configure the current truncation so that the full contents of a log can be POSTed to Healthchecks instead of the default 10kb. #### Other notes / implementation ideas This is a continuation of #241 and #249, to allow the full body of the borgmatic output to get to Healthchecks.

witten commented

2020-04-29 17:18:09 +00:00

Makes sense! I could see having a configuration option for setting the truncation size (or disabling truncation altogether).

I think the biggest challenge here is not implementing the configurable truncation, but rather figuring out how to fit the option into the current borgmatic config file schema.. The current schema looks like:

hooks:
    healthchecks: https://hc-ping.com/your-uuid-here

So the best option may be to change the single scalar value to a map. So something like:

hooks:
    healthchecks:
        url: https://hc-ping.com/your-uuid-here
        body_size: 1234

And then put in some code to make loading of the previous schema backwards-compatible. (There's precedent for that.)

Makes sense! I could see having a configuration option for setting the truncation size (or disabling truncation altogether). I think the biggest challenge here is not implementing the configurable truncation, but rather figuring out how to fit the option into the current borgmatic config file schema.. The current schema looks like: ```yaml hooks: healthchecks: https://hc-ping.com/your-uuid-here ``` So the best option may be to change the single scalar value to a map. So something like: ```yaml hooks: healthchecks: url: https://hc-ping.com/your-uuid-here body_size: 1234 ``` And then put in some code to make loading of the previous schema backwards-compatible. (There's precedent for that.)

nightah commented

2020-04-30 00:43:27 +00:00

Yep that looks great to me!

witten added the

design finalized

label 2020-04-30 04:22:52 +00:00

witten referenced this issue from a commit

2022-05-24 19:24:07 +00:00

Add Healthchecks monitoring hook "ping_body_limit" option to configure how many bytes of logs to send to the Healthchecks server (#294).

witten commented

2022-05-24 19:25:52 +00:00

Well, it took a while to get to it (my apologies!), but this is now implemented in master. There's a new ping_body_limit option under healthchecks: that you can set to a number of bytes or even set to zero to disable truncation. Thanks for suggesting this!

Well, it took a while to get to it (my apologies!), but this is now implemented in master. There's a new `ping_body_limit` option under `healthchecks:` that you can set to a number of bytes or even set to zero to disable truncation. Thanks for suggesting this!

witten closed this issue

2022-05-24 19:25:52 +00:00

witten commented

2022-05-24 23:24:54 +00:00

Released in borgmatic 1.6.1!

Sign in to join this conversation.