Skip to content

Read the HeaderValueOption if passed and overwrite or append accordingly - #4246

Open
Pushpalanka wants to merge 1 commit into
masterfrom
read-header-value-option-overwrite-or-append
Open

Read the HeaderValueOption if passed and overwrite or append accordingly#4246
Pushpalanka wants to merge 1 commit into
masterfrom
read-header-value-option-overwrite-or-append

Conversation

@Pushpalanka

Copy link
Copy Markdown
Collaborator

Current behavior is kept as default and no change to the existing behavior when option is not passed.

Keep the behavior consistent with Envoy external authorization header behavior, as taken to attention with GHSA-wvv5-jv5r-xq52

@Pushpalanka Pushpalanka added the minor no risk changes, for example new filters label Sep 4, 2026
@Pushpalanka
Pushpalanka force-pushed the read-header-value-option-overwrite-or-append branch from 2784abc to 6924997 Compare September 4, 2026 11:37
@szuecs

szuecs commented Sep 4, 2026

Copy link
Copy Markdown
Member

@Pushpalanka see make fmt

…gly.

Current behavior is kept as default and no change to the existing behavior when option is not passed.

Signed-off-by: Pushpalanka Jayawardhana <pushpalankajaya@gmail.com>
@Pushpalanka
Pushpalanka force-pushed the read-header-value-option-overwrite-or-append branch from 6924997 to cc59947 Compare September 4, 2026 15:04
@zalando-robot

Copy link
Copy Markdown

Docker image "registry-write.opensource.zalan.do/teapot/skipper:cc599475a735f102ef2f7e49e47064ca4212ea90" is not based on an approved base image. Any production deployment relying on this image will be blocked.

To create a compliant Docker image of your application, you must reference an allowed Docker image as its base image in your Dockerfile. This base image must come from the Zalando Container Registry namespace library and use a recommended version as listed in the documentation.

@Pushpalanka

Copy link
Copy Markdown
Collaborator Author
  • pipeline

Thanks a lot for the hint.

@Pushpalanka

Copy link
Copy Markdown
Collaborator Author

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

minor no risk changes, for example new filters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants