Skip to content
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

cannot delete topic/group perms in acl config #3281

Closed
yuz10 opened this issue Aug 20, 2021 · 0 comments
Closed

cannot delete topic/group perms in acl config #3281

yuz10 opened this issue Aug 20, 2021 · 0 comments

Comments

@yuz10
Copy link
Member

yuz10 commented Aug 20, 2021

The issue tracker is ONLY used for bug report(feature request need to follow RIP process). Keep in mind, please check whether there is an existing same report before your raise a new one.

Alternately (especially if your communication is not a bug report), you can send mail to our mailing lists. We welcome any friendly suggestions, bug fixes, collaboration and other improvements.

Please ensure that your bug report is clear and that it is complete. Otherwise, we may be unable to understand it or to reproduce it, either of which would prevent us from fixing the bug. We strongly recommend the report(bug report or feature request) could include some hints as the following:

BUG REPORT

  1. Please describe the issue you observed:
  • What did you do (The steps to reproduce)?

In mqadmin updateAclConfig -n localhost:9876 -a abcdefg -s abcdefg -c RaftCluster -g groupa=SUB
sets group groupPerms to groupa=SUB
and then wants to delete groupPerms using
mqadmin updateAclConfig -n localhost:9876 -a abcdefg -s abcdefg -c RaftCluster -g ""

  • What did you expect to see?
    groupPerms is empty

  • What did you see instead?
    cannot delete groupPerms, its still groupa=SUB

  1. Please tell us about your environment:

  2. Other information (e.g. detailed explanation, logs, related issues, suggestions how to fix, etc):

FEATURE REQUEST

  1. Please describe the feature you are requesting.

  2. Provide any additional detail on your proposed use case for this feature.

  3. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have). Are you currently using any workarounds to address this issue?

  4. If there are some sub-tasks using -[] for each subtask and create a corresponding issue to map to the sub task:

@duhenglucky duhenglucky added this to the 4.9.2 milestone Aug 21, 2021
vongosling pushed a commit that referenced this issue Sep 14, 2021
…te address(#3128) (#3280)

* fix fail to remove topic/group permissions and global white address of acl config

* update acl config for all brokers

* refactor rename UtilAll.string2List and list2String

* fix fail to remove whiteRemoteAddress of acl config
@yuz10 yuz10 closed this as completed Sep 15, 2021
GenerousMan pushed a commit to GenerousMan/rocketmq that referenced this issue Aug 12, 2022
…al white address(apache#3128) (apache#3280)

* fix fail to remove topic/group permissions and global white address of acl config

* update acl config for all brokers

* refactor rename UtilAll.string2List and list2String

* fix fail to remove whiteRemoteAddress of acl config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants