-
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
[#12423] allow GetTopicsOfNamespace
op with consume
permission
#12600
[#12423] allow GetTopicsOfNamespace
op with consume
permission
#12600
Conversation
@freeznet:Thanks for your contribution. For this PR, do we need to update docs? |
@freeznet:Thanks for providing doc info! |
@freeznet:Thanks for providing doc info! |
Fixes #12423 ### Motivation Regex subscription requires to get the topics list of given namespace with GetTopicsOfNamespace request, but this request requires tenant admin permission which will block the regex consumers who only have consume permission. ### Modifications This PR added the consume permission check for GetTopicsOfNamespace, which allows consumers get the topics list with consume permission. (cherry picked from commit 7e078aa)
Fixes #12423 ### Motivation Regex subscription requires to get the topics list of given namespace with GetTopicsOfNamespace request, but this request requires tenant admin permission which will block the regex consumers who only have consume permission. ### Modifications This PR added the consume permission check for GetTopicsOfNamespace, which allows consumers get the topics list with consume permission. (cherry picked from commit 7e078aa)
Fixes apache#12423 ### Motivation Regex subscription requires to get the topics list of given namespace with GetTopicsOfNamespace request, but this request requires tenant admin permission which will block the regex consumers who only have consume permission. ### Modifications This PR added the consume permission check for GetTopicsOfNamespace, which allows consumers get the topics list with consume permission.
Updated the label, as this commit is included in the 2.9.0 release. See #13133 for more information. |
The apache/pulsar#12600 changed behavior. Before this change, only tenant admin allow to list topics, after apache/pulsar#12600, it allows consume permission list topics.
The apache/pulsar#12600 changed behavior. Before this change, only tenant admin allow to list topics, after apache/pulsar#12600, it allows consume permission list topics.
It seems that this patch is not present in v3.2.0 |
Fixes #12423
Motivation
Regex subscription requires to get the topics list of given namespace with
GetTopicsOfNamespace
request, but this request requires tenant admin permission which will block the regex consumers who only haveconsume
permission.Modifications
This PR added the
consume
permission check forGetTopicsOfNamespace
, which allows consumers get the topics list withconsume
permission.Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Does this pull request potentially affect one of the following parts:
If
yes
was chosen, please highlight the changesDocumentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?
doc-required
(If you need help on updating docs, create a doc issue)
no-need-doc
(Please explain why)
doc
(If this PR contains doc changes)