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