sunxi92 commented on issue #3922: URL: https://github.com/apache/rocketmq/issues/3922#issuecomment-1057988228
> @zergduan @sunxi92 I will check the following list before submitting my pr, please feel free to tell me if any is missing: Test cases should cover the following circumstances: > > 1. Only `conf/plain_acl.yml` exists; > 2. Only `/conf/acl/plain_acl.yml` exists: In my pr, an empty `conf/plain_acl.yml` would be created, so it is the same as the next circumstance; > 3. Both `conf/plain_acl.yml` and `/conf/acl/plain_acl.yml` exists > > In each of the above circumstance, check: > > * use admin command to view and modify ACL, including global white addresses and account authorities; > * modify ACL config file directly, and the acl would be refreshed correctly > * check producing and consuming messages with ACL I think we can consider the scenarion that the acl configuration only have globalWhiteRemoteAddresses or accounts. In addition, we can explain in the documentation what the tools.yml file does and how you need to set the user in the tools.yml file to admin in the acl configuration. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
