RockteMQ-AI commented on issue #2422: URL: https://github.com/apache/rocketmq-dashboard/issues/2422#issuecomment-5351224043
**Issue Evaluation** Category: `bug` | Status: **Confirmed** The reported issue has been verified against the current codebase. **Root Cause:** `MetadataService.deleteTopic(String instanceId, String name)` at line 78-79 does not call `requireName(name, "topic name")` unlike sibling methods (`getTopicRoutes`, `getTopicConsumers`, `getConsumerGroup`, etc.). This means: - No validation: null/blank names are not rejected with 400 - No trimming: whitespace-padded names like `" orders "` reach providers untrimmed - Inconsistency: violates the service boundary pattern used by all other operations **Impact:** Apache/Tencent/Aliyun providers receive untrimmed topic names; inconsistent behavior across service methods. **Severity:** Medium - data inconsistency risk, but no data loss. **Note:** PR #2423 already addresses this fix. --- *Automated evaluation by github-manager* -- 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]
