Update: As per #2006 Empty body being redacted is confusing

[redcinelli]

As per #2006

When the body is empty, It feels surprising to set it to redacted.
Even more so when we have the CaptureBody: true.

[github-actions]

👋 @redcinelli Thanks a lot for your contribution!

It may take some time before we review a PR, so even if you don’t see activity for some time, it does not mean that we have forgotten about it.

Every once in a while we go through a process of prioritization, after which we are focussing on the tasks that were planned for the upcoming milestone. The prioritization status is typically reflected through the PR labels. It could be pending triage, a candidate for a future milestone, or have a target milestone set to it.

[Mpdreamz]

Thanks for tackling this @redcinelli and apologies for taking forever to get around to it.

I've amended the PR slightly to forcibly set the body to redacted if the config is set to off and made sure the default is not an empty string because we attempt to read the body multiple times at various stages in the pipeline.

LGTM but my colleague @stevejgordon will need to review now that I also added code 😸

[stevejgordon]

LGTM, thanks.