yyqdbngt opened a new pull request, #3098: URL: https://github.com/apache/rocketmq-dashboard/pull/3098
### Summary Cover the input normalization paths of `BrokerConfigDiffService` (complementary file to the drift tests): - blank `clusterId` (and the blank cluster on the instance overload) is rejected with the 400 `cluster is required`; - the instance-scoped `compare` trims both ids and routes through `getCluster(clusterId, instance)` and `getBrokerConfig(addr, instance)`; - a blank instance id falls back to the unscoped single-argument lookup (and the no-broker 409) without any broker-config read. ### Why The two public `compare` overloads normalize ids before lookup; locking the blank/instance routing down prevents regressions in request handling. ### Testing - `mvn -f server/pom.xml -Dtest=BrokerConfigDiffInputTest test` passes: 3/3. -- 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]
