Improve Dict log for reqHeaders & resHeaders

[roman-wb]

Hi, @ReneWerner87!

What do you think about it?

1. Improved request headers log:
   Was:
   {"method":"POST"," header_key":"header_value"}
   Became with sub dict:
   {"method":"POST", "reqHeaders": {"header_key":"header_value"}}
2. Added response headers log (with field FieldResHeaders = "resHeaders"):
   {"method":"POST", "resHeaders": {"header_key":"header_value"}}

[ReneWerner87]

thx for the PR, but iam not really an expert for this MW
i will tag someone with more exp

[ReneWerner87]

@iredmail @raphielscape @dotX12 what do you think?

[raphielscape]

Seems like a breaking change for me, but overall it's fine

[dotX12]

This will affect users who have header filters configured, for example, dashboards in Grafana, i.e. this is a serious change.

But in the long run it looks sensible, I’m all for it.
LGTM...

[ReneWerner87]

so it's a breaking change
would it be possible to design this change without breaking?

[dotX12]

so it's a breaking change would it be possible to design this change without breaking?

I would add a flag so that users can choose the format for listing headers.
Something like:

HeadersFormat: rolled | unrolled

maybe

HeadersFormat : flat | object

or

WrapHeaders: true / false

In this case, we won't break anything :)

[roman-wb]

Thx, without breaking done)