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

   Fixes #684
   
   ### Problem
   
   `MessageProviderStub` is wired as the production `MessageProvider`, but 
`queryMessages` returns an empty list and `getMessageTrace` returns an empty 
trace. A successful empty response can mislead operators into thinking no 
message or trace exists, when in fact no real provider is connected.
   
   ### Fix
   
   Until a real message provider is connected, both operations now fail 
explicitly with a structured `501 Not Implemented` `BusinessException` (mapped 
by `GlobalExceptionHandler` to a clean `Result.error(501, ...)`), matching the 
issue's expected behavior. Added `MessageProviderStubTest`.
   
   ### Verification
   
   `mvn test -Dtest=MessageProviderStubTest` on JDK 21: `Tests run: 2, 
Failures: 0, Errors: 0, Skipped: 0`.
   
   ### Diff
   
   2 files changed, +56 / -10.


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