Skip to content

Forward the caller's identity to virtual workspaces - #4310

Merged
kcp-ci-bot merged 2 commits into
kcp-dev:mainfrom
mjudeikis:identity.forwarding
Aug 4, 2026
Merged

Forward the caller's identity to virtual workspaces#4310
kcp-ci-bot merged 2 commits into
kcp-dev:mainfrom
mjudeikis:identity.forwarding

Conversation

@mjudeikis

Copy link
Copy Markdown
Contributor

Summary

The shard proxies virtual resources to the virtual workspace over a connection that authenticates as the shard, so the VW authorized the shard rather than the user. Stamp the request-header identity on the forwarded request instead, clearing any inbound copies first.

What Type of PR Is This?

/kind documentation
/kind feature

Related Issue(s)

Fixes #

Release Notes

Add identity forward to virtual workspaces

The shard proxies virtual resources to the virtual workspace over a
connection that authenticates as the shard, so the VW authorized the
shard rather than the user. Stamp the request-header identity on the
forwarded request instead, clearing any inbound copies first.

Signed-off-by: Mangirdas Judeikis <mangirdas@judeikis.lt>
On-behalf-of: SAP <mangirdas.judeikis@sap.com>
@kcp-ci-bot kcp-ci-bot added release-note Denotes a PR that will be considered when it comes time to generate release notes. kind/documentation Categorizes issue or PR as related to documentation. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/feature Categorizes issue or PR as related to a new feature. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Aug 4, 2026

@ntnn ntnn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

/lgtm
/approve

@kcp-ci-bot kcp-ci-bot added the lgtm Indicates that a PR is ready to be merged. label Aug 4, 2026
@kcp-ci-bot

Copy link
Copy Markdown
Contributor

LGTM label has been added.

DetailsGit tree hash: 4d57becc897572e2c248340264f387f9f9e1c465

@kcp-ci-bot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ntnn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kcp-ci-bot kcp-ci-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 4, 2026
@kcp-ci-bot
kcp-ci-bot merged commit b9b9893 into kcp-dev:main Aug 4, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has signed the DCO. kind/documentation Categorizes issue or PR as related to documentation. kind/feature Categorizes issue or PR as related to a new feature. lgtm Indicates that a PR is ready to be merged. release-note Denotes a PR that will be considered when it comes time to generate release notes. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants