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

   ### Summary
   Require the consumer group and client id in the stack-diagnostics lookup:
   
   - `getConsumerStack` trims both inputs and rejects blank values with a clear 
400 (`consumer group is required` / `consumer client id is required`) before 
any proxy or broker probe.
   
   ### Why
   A blank group/client previously flowed into the probes and surfaced as a 
confusing "consumer not online" style failure; an explicit 400 makes the 
contract clear at the boundary.
   
   ### Testing
   - `mvn -f server/pom.xml -Dtest=RocketMQConsumerDiagnosticsProviderTest 
test` passes: 12/12 (11 existing + 1 new covering both blank guards).
   


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