tju-yxq commented on issue #1798: URL: https://github.com/apache/rocketmq-dashboard/issues/1798#issuecomment-5269465649
## Clarification after re-checking The original issue text was too close to a live reproduction report. The current evidence is code inspection plus focused service-layer unit coverage. The bug being tracked here is specifically that ACL 1.0 plain access config did not validate a non-blank `whiteRemoteAddress` before delegating to the repository, while ACL 2.0 `whiteSet` validation already used `IpRangeMatcher`. Additional validation added in PR #1799: ```bash mvn -DskipTests=false -Dtest=AclServiceTest#createAndUpdatePlainAccessConfigShouldRejectInvalidWhiteRemoteAddress+createAndUpdatePlainAccessConfigShouldAcceptValidWhiteRemoteAddress test ``` Result: `BUILD SUCCESS`, `Tests run: 2, Failures: 0, Errors: 0, Skipped: 0`. The test proves that invalid values are rejected before persistence and valid CIDR values still delegate to the repository. -- 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]
