X-LightYear opened a new issue, #7297:
URL: https://github.com/apache/rocketmq-dashboard/issues/7297

   ## Problem
   
   The `rmq.message.redelivery_dlq` preview shows the caller-provided 
`targetTopic` without normalization. The actual `DLQService.resendMessages` 
path trims surrounding whitespace before sending, so a dry-run plan can display 
a different target topic from the mutation that will execute.
   
   ## Reproduction
   
   Call the MCP tool in dry-run mode with:
   
   ```json
   {
     "instanceId": "instance-a",
     "groupName": "group-a",
     "targetTopic": " orders-retry ",
     "dry_run": true
   }
   ```
   
   ## Expected behavior
   
   The preview should show `orders-retry`, matching the normalized target used 
by the service on execution.
   
   ## Actual behavior
   
   The preview contains ` orders-retry ` with whitespace, while `DLQService` 
sends to `orders-retry`.
   
   ## Fix
   
   Normalize the target topic in the handler's preview projection, preserving 
the existing `ORIGINAL_TOPIC` fallback and execution behavior.
   
   ## Testing
   
   - Pre-fix deterministic regression failed for a whitespace-padded target 
topic.
   - `MessageRedeliveryDlqToolHandlerTest`: 9 passed.
   - Checkstyle: 0 violations.
   - `git diff --check`: passed.
   


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