RockteMQ-AI commented on issue #1802: URL: https://github.com/apache/rocketmq-dashboard/issues/1802#issuecomment-5254291093
**Issue Evaluation** Category: `bug` | Status: **Confirmed** The reported missing validation has been verified in the DLQ message query flow. **Root Cause:** `MessageQuery` accepts `begin` and `end` timestamps without validating that `begin < end`. When `begin > end`, `searchOffset` returns inverted results and the scan silently returns zero messages. **Impact:** Users querying DLQ messages with inverted time ranges get empty results without any error indication, making debugging difficult. **Severity:** Low-Medium — doesn't cause data corruption but leads to confusing empty results. The regular message query already has this validation. **Suggested Fix:** Add validation in `queryDlqMessageByPage()` or the controller to check `begin < end` and throw a `ServiceException` if invalid, matching the existing pattern in message query. An automated fix proposal can be generated. Reply `/approve` to proceed with PR generation. --- *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]
