yyqdbngt opened a new issue, #2250:
URL: https://github.com/apache/rocketmq-dashboard/issues/2250

   ## Problem
   
   The real client-connections endpoint requires an `instanceId`, but the mock 
service ignores it and starts from the complete global mock inventory. 
Switching Studio instances in mock mode therefore shows connections belonging 
to other instances, and even a missing or unknown instance can expose unrelated 
data.
   
   This makes the mock/demo workflow disagree with the real endpoint contract 
and can hide instance-scoping regressions in page tests.
   
   ## Expected behavior
   
   - Require a non-blank instance ID in mock mode, matching the real request 
contract.
   - Return only connections owned by the selected mock instance.
   - Return an empty inventory for an unknown instance.
   - Apply optional cluster and client-type filters after instance isolation.
   - Continue returning defensive copies of mock records.
   
   ## Proposed scope
   
   Declare the mock instance-to-cluster ownership used by the fixture, enforce 
it in the connection service, and add service coverage for isolation, 
missing/unknown instances, combined filters, and defensive copying.
   
   ## Duplicate check
   
   Searched open and closed issues and pull requests for mock client connection 
instance scoping and `instanceId` handling. No matching implementation was 
found, and no current pull request targeting `rocketmq-studio` changes the 
proposed mock, service, or test files.
   


-- 
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]

Reply via email to