Skip to content

Prevent Kube proxy from set the default Kube impersonation headers#32839

Merged
tigrato merged 1 commit intomasterfrom
tigrato/fix-default-kube-imp-headers
Oct 2, 2023
Merged

Prevent Kube proxy from set the default Kube impersonation headers#32839
tigrato merged 1 commit intomasterfrom
tigrato/fix-default-kube-imp-headers

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Oct 2, 2023

This PR removes the ability of the Kubernetes Proxy to set the default values of Kubernetes Impersonation headers. This task is delegated to the kubernetes_service or kubernetes legacy proxy that handles the request.

This is required because under certain contiditons it might have requests that failed because the values were improperly defined.

This PR removes the ability of the Kubernetes Proxy to set the default
values of Kubernetes Impersonation headers. This task is delegated to
the kubernetes_service or kubernetes legacy proxy that handles the
request.

This is required because under certain contiditons it might have
requests that failed because the values were improperly defined.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
Copy link
Copy Markdown
Contributor

@strideynet strideynet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thanks for including test cases.

@tigrato tigrato enabled auto-merge October 2, 2023 13:13
@tigrato tigrato added this pull request to the merge queue Oct 2, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 2, 2023
@tigrato tigrato added this pull request to the merge queue Oct 2, 2023
@tigrato tigrato removed this pull request from the merge queue due to a manual request Oct 2, 2023
@tigrato tigrato added this pull request to the merge queue Oct 2, 2023
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Oct 2, 2023
@tigrato tigrato added this pull request to the merge queue Oct 2, 2023
Merged via the queue into master with commit 955331b Oct 2, 2023
@tigrato tigrato deleted the tigrato/fix-default-kube-imp-headers branch October 2, 2023 14:44
@public-teleport-github-review-bot
Copy link
Copy Markdown

@tigrato See the table below for backport results.

Branch Result
branch/v14 Create PR

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants