yyqdbngt opened a new pull request, #3417: URL: https://github.com/apache/rocketmq-dashboard/pull/3417
### Summary `ClientService` guards required identifiers before delegating to the client provider and passes provider results through unchanged. The suite covered blank-value guards but not the `null` identifier or the non-empty passthrough on the nameserver-address path. ### Changes - A `null` instance id is rejected with `400 instanceId is required` before the provider is touched - The nameserver-address lookup returns provider results unchanged after trimming and blanking optional filters ### Testing `mvn test -Dtest=ClientServiceTest` passes: 7 tests, 0 failures (up from 5). -- 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]
