-
Notifications
You must be signed in to change notification settings - Fork 3.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[improve][sec] Align some namespace level policy authorisation check #21640
[improve][sec] Align some namespace level policy authorisation check #21640
Conversation
Some of our namespace level policy already has namespace operation, and the read method has a corresponding authorisation check, but write does not have it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Cherry-pick to branch-2.11 by #22248 |
@Technoboy- please tag the release in this PR to track the cherry-picking. Just wondering if this should go to 3.1.x and 3.0.x too if it's going to 2.11.x . |
Oh, sorry, miss this comment. |
…pache#21640) (cherry picked from commit 7fdfbf4)
…pache#21640) (cherry picked from commit 7fdfbf4)
Motivation
Some of our namespace level policy already has namespace operation, and the read method has a corresponding authorisation check, but write does not have it.
Modifications
Verifying this change
Documentation
doc
doc-required
doc-not-needed
doc-complete