caigy commented on issue #3922: URL: https://github.com/apache/rocketmq/issues/3922#issuecomment-1057962258
@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 -- 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]
