RockteMQ-AI commented on issue #2454: URL: https://github.com/apache/rocketmq-dashboard/issues/2454#issuecomment-5358511090
**Issue Evaluation** Category: `bug` | Status: **Confirmed** Verified in `ProducerConnectionSummaryVO.java`. The `hasText()` method (line 143) filters out `"null"` (case-sensitive) but not `"NULL"` or other case variants. The `duplicateClientIds()` method (line 94) uses the same `hasText()` filter, so uppercase `"NULL"` passes through and multiple connections with `"NULL"` client IDs are counted as duplicates. This creates a contradiction: `hasIncompleteMetadata()` flags `"null"` as missing, but `duplicateClientIds()` flags `"NULL"` as a real duplicate — the same semantic placeholder produces different diagnostics depending on case. **Root Cause:** `hasText()` uses case-sensitive comparison for the `"null"` placeholder. **Impact:** Contradictory missing vs duplicate diagnostics for null-placeholder client IDs. **Severity:** medium The fix should use case-insensitive null-placeholder detection in `hasText()`. --- *Automated evaluation by RockteMQ-AI* -- 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]
