RockteMQ-AI commented on issue #566: URL: https://github.com/apache/rocketmq-dashboard/issues/566#issuecomment-5771469778
**Issue Evaluation** Category: `bug` | Status: **Confirmed** This is a valid bug report. The audit log API silently ignores invalid date parameters instead of rejecting them with a clear error. **Root Cause:** Date parsing failures are caught and converted to an empty filter, masking the error from the caller. **Impact:** Users get empty results without understanding why — invalid dates produce the same response as "no data in range." **Severity:** Medium — incorrect API behavior that hinders debugging and can lead to data being silently missed. **Proposed Fix (from issue):** 1. Convert date parsing failures to `BusinessException` with code 400 2. Validate date range (startDate ≤ endDate) before querying 3. Require `YYYY-MM-DD` format explicitly 4. Preserve inclusive full-day filtering for valid ranges The proposed fix is sound. A PR (#567) has been filed to address this. --- *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]
