123123213weqw opened a new pull request, #752:
URL: https://github.com/apache/rocketmq-dashboard/pull/752

   Fixes #696
   
   ### Problem
   
   `ClientProviderStub` is wired as the production `ClientProvider` but returns 
hard-coded sample connections (`producer-001`, `consumer-001`, `consumer-002`). 
The client connection page should not show production-looking fake clients when 
no real client provider is configured — it can mislead operators during 
validation.
   
   ### Fix
   
   Until a real Remoting/gRPC client provider is connected, `findConnections` 
now fails explicitly with a structured `501 Not Implemented` 
`BusinessException` (mapped by `GlobalExceptionHandler` to `Result.error(501, 
...)`), matching the issue's expected behavior. Added `ClientProviderStubTest`.
   
   ### Verification
   
   `mvn test -Dtest=ClientProviderStubTest` on JDK 21: `Tests run: 2, Failures: 
0, Errors: 0, Skipped: 0`.
   
   ### Diff
   
   2 files changed.


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