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

   ### Summary
   Trim the DLQ consumer group name before it reaches the provider:
   
   - `resendMessages`, `exportMessages`, `listMessages`, 
`resendSelectedMessages` and `exportExcel` now resolve the group through 
`requireGroupName` (rejects null/blank with the existing 400, then trims) and 
pass the trimmed value to the provider.
   
   ### Why
   A group name with accidental leading/trailing whitespace previously passed 
the `hasText` validation but then missed every broker lookup (RocketMQ group 
names never contain surrounding spaces), producing confusing "group not 
found"-style failures. Trimming at the service boundary makes the five entry 
points consistent with the already-trimmed search path in `listDLQGroups`.
   
   ### Testing
   - `mvn -f server/pom.xml -Dtest=DLQServiceTest test` passes: 15/15 (14 
existing + 1 new asserting the provider receives the trimmed group name).
   


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