zhaohai666 opened a new pull request, #656:
URL: https://github.com/apache/rocketmq-dashboard/pull/656

   ## Summary
   - Add generic `Result<T>` API response wrapper with `ok()`/`error()` factory 
methods for standardized REST responses
   - Add `BusinessException` with integer error code for domain-level error 
handling
   - Add `GlobalExceptionHandler` (`@RestControllerAdvice`) to handle 
BusinessException, PrometheusException, validation errors 
(MethodArgumentNotValidException), and malformed requests 
(HttpMessageNotReadableException)
   - Include `PrometheusException` as a dependency for the metrics module 
(PR-02)
   
   ## Test plan
   - [ ] Verify `Result.ok()` and `Result.error()` return correct 
status/data/errMsg fields
   - [ ] Verify GlobalExceptionHandler catches BusinessException and returns 
proper HTTP 400 response
   - [ ] Verify GlobalExceptionHandler catches validation errors and returns 
proper response
   - [ ] Verify PrometheusException carries HTTP status code correctly


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