RockteMQ-AI commented on issue #1295:
URL: 
https://github.com/apache/rocketmq-dashboard/issues/1295#issuecomment-5231447005

   **Issue Evaluation**
   
   Category: `bug` | Status: **Confirmed**
   
   The reported boundary violation is valid. `RuntimeAdminClientResolver` in 
the `rocketmq-studio` branch validates only instance ID and endpoint, without 
checking the vendor/instance type. This allows cloud-backed instances to reach 
`MqAdminExtFactory` through the Remoting protocol path, which is incorrect.
   
   **Root Cause:** Missing vendor type check in `RuntimeAdminClientResolver` — 
cloud instances should be rejected from Remoting admin operations.
   **Impact:** Any non-Apache instance selected in the dashboard could trigger 
incorrect protocol handling and confusing errors.
   **Severity:** medium
   
   A fix is already proposed in PR #1296 (`fix: reject cloud instances from 
Remoting admin resolution`), which adds the `RuntimeAdminClientResolverTest` 
for regression coverage.
   
   ---
   *Automated evaluation by github-manager-bot*


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