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
Fix #2265: Refactor SubjectAccessReview API #2400
Conversation
a8d52a9
to
f59bbb8
Compare
kubernetes-client/src/main/java/io/fabric8/kubernetes/client/CreateOnlyResourceOperations.java
Outdated
Show resolved
Hide resolved
...shift-client/src/main/java/io/fabric8/openshift/client/OpenshiftSubjectAccessOperations.java
Outdated
Show resolved
Hide resolved
f59bbb8
to
09767fe
Compare
this.subjectAccessApiGroupName = apiGroupName; | ||
this.subjectAccessApiGroupVersion = apiGroupVersion; | ||
this.plural = plural; |
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.
This is redundant, previous statement already takes care of this. I think you can also declare the fields final.
|
||
public CreateOnlyResourceOperationsImpl(OkHttpClient client, Config config, String apiGroupName, String apiGroupVersion, String plural, Class<T> subjectAccessRequestClass) { | ||
this(new OperationContext().withOkhttpClient(client).withConfig(config), apiGroupName, apiGroupVersion, plural, subjectAccessRequestClass); | ||
this.subjectAccessRequestClass = subjectAccessRequestClass; |
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.
+ Added new entrypoint authorization() into DSL which only allows create() operations + Removed old subjectAccessReviewAuth() API This should also fix fabric8io#2191
TokenReview also are create only resources like SubjectAccess* operations in which same object is returned in response with status object modified. It's support can be added easily with current support added for SubjectAccessReview.
09767fe
to
653f98f
Compare
SonarCloud Quality Gate failed. 0 Bugs |
[merge] |
Fixes #2265
This should also fix #2191
Description
Type of change
test, version modification, documentation, etc.)
Checklist