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

   ## Summary
   
   - preserve structured `BusinessException` status codes and messages from 
instance and credential resolution
   - keep unexpected Admin failures wrapped by the existing reset-offset 500 
response
   - retain failed audit records for both error paths
   - add regression coverage for a resolver 404 and an unexpected Broker/Admin 
failure
   
   Closes #1994
   
   ## Verification
   
   - failure proof before the production fix: the new resolver test expected 
`Instance not found: missing-instance` with code 404 but received `Failed to 
reset offset: Instance not found: missing-instance` with code 500
   - `JAVA_HOME=$(/usr/libexec/java_home -v 21) PATH="$JAVA_HOME/bin:$PATH" mvn 
-f server/pom.xml -Dtest=RocketMQAdminClientImplTest test` (21 tests passed)
   - `JAVA_HOME=$(/usr/libexec/java_home -v 21) PATH="$JAVA_HOME/bin:$PATH" mvn 
-f server/pom.xml -DskipTests package`
   - `git diff --check`
   


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