Skip to content

[Breaking Change Guarded by a Flag] fix: restore CSR nodeclient DataAction granularity - #28

Open
Wei Weng (weng271190436) wants to merge 1 commit into
Azure:masterfrom
weng271190436:weiweng/fix-csr-subresource-action-collapse
Open

[Breaking Change Guarded by a Flag] fix: restore CSR nodeclient DataAction granularity#28
Wei Weng (weng271190436) wants to merge 1 commit into
Azure:masterfrom
weng271190436:weiweng/fix-csr-subresource-action-collapse

Conversation

@weng271190436

@weng271190436 Wei Weng (weng271190436) commented Aug 26, 2026

Copy link
Copy Markdown
Member

Summary

Reintroduces the security fix from #12 after its temporary rollback in #23, using Guard's current sensitive-subresource mapping framework.

Kubernetes authorizes kubelet client CSR auto-approval through the virtual certificatesigningrequests/nodeclient subresource. Guard currently collapses this request into the broader certificatesigningrequests/write DataAction.

This PR adds the dedicated mapping behind a default-off compatibility flag:

--azure.enforce-csr-nodeclient-data-action=false

When enabled, Guard maps the request to:

certificates.k8s.io/certificatesigningrequests/nodeclient/action

When disabled, Guard preserves the current certificatesigningrequests/write mapping. This allows the Guard image to ship before enforcement and provides an immediate rollback switch without requiring an image rollback.

@weng271190436
Wei Weng (weng271190436) force-pushed the weiweng/fix-csr-subresource-action-collapse branch 2 times, most recently from 817d2d2 to 023147f Compare August 27, 2026 14:36
@weng271190436
Wei Weng (weng271190436) force-pushed the weiweng/fix-csr-subresource-action-collapse branch from 023147f to d40d4a2 Compare August 27, 2026 14:51
@weng271190436 Wei Weng (weng271190436) changed the title [Breaking Change] fix: restore CSR nodeclient DataAction granularity [Breaking Change Guarded by a Flag] fix: restore CSR nodeclient DataAction granularity Aug 27, 2026
@weng271190436
Wei Weng (weng271190436) marked this pull request as ready for review August 27, 2026 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant