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

   ## What is the purpose of the change
   
   Fix #1994.
   
   RocketMQAdminClientImpl.resetOffset() caught every exception and replaced it 
with BusinessException(500, ...). This also caught structured control-plane 
errors raised before any Broker call, including 400 for a missing endpoint, 404 
for a missing instance and 422 for an unconfigured admin credential. The API 
therefore reported an internal server error instead of the actual problem.
   
   ## Brief changelog
   
   - RocketMQAdminClientImpl.resetOffset: rethrow BusinessException as-is so 
structured validation errors keep their code and message; wrap only unexpected 
exceptions in 500
   
   ## How was this patch verified
   
   - Code review: runtimeAdminClientResolver throws BusinessException(400/422) 
which now propagates unchanged
   - `git diff --check` clean
   


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