Make Flow ID header configurable about skipper HOT 3 CLOSED

As a workaround you may try adding default filters like so:

-default-filters-prepend='copyRequestHeader("X-Flux-Id", "X-Flow-Id") -> dropHeader("X-Flux-Id")'
-default-filters-append='copyRequestHeader("X-Flow-Id", "X-Flux-Id") -> dropHeader("X-Flow-Id")'

from skipper.

Tested the workaround today. It correctly sets the new header, X-Flux-Id, but the field flow-id in the access logs stays empty.
My guess is that this happens because the write of the access log is deferred here and therefore happens after all filters have been processed already.

While looking into this, I noticed that it is possible to add custom fields to the access log through a filter via the AccessLogAdditionalDataKey of the state bag. A custom filter can set its own value of the field flow-id, which is an acceptable workaround for this issue.

Closing the issue for now.

from skipper.

@wndhydrnt Thank your for the feedback. I think you are right regarding access logs, you may try default-filters-append without dropHeader("X-Flow-Id")' with a caveat that backend will see X-Flow-Id header which might be not a big deal really.

from skipper.