linkerd / linkerd2

Ultralight, security-first service mesh for Kubernetes. Main repo for Linkerd 2.x.
https://linkerd.io
Apache License 2.0
10.47k stars 1.26k forks source link

Make logHTTPHeaders optional #12747

Closed adleong closed 1 week ago

adleong commented 1 week ago

The proxy.logHTTPHeaders Helm variable supports two values: "insecure" and "off". However, if this value is absent entirely, validation will fail because it it not one of the supported values. This can cause issues with backwards compatibility.

We make this value optional by adding "" as a legal value during validation. Leaving this value unset is equivalent to "off" which is the default value.