RockteMQ-AI commented on issue #10904: URL: https://github.com/apache/rocketmq/issues/10904#issuecomment-5267869823
**Issue Evaluation** Category: `bug` | Status: **Confirmed** The reported issue has been verified against the current codebase. **Root Cause:** In `ProducerProcessor.java` (lines 214, 220), `Integer.valueOf(reconsumeTimes)` and `Integer.valueOf(maxReconsumeTimes)` are called without defensive parsing. If a message property contains a non-numeric value, a `NumberFormatException` propagates up and aborts the entire `sendMessage` request — affecting all messages in the batch, not just the malformed one. **Impact:** Proxy message forwarding path. A single malformed retry property can block all messages in a send request. **Severity:** Medium — availability impact on the Proxy forwarding path. **Suggested Fix:** Wrap the `Integer.valueOf` calls with try-catch and fall back to a safe default (e.g., `0` for reconsumeTimes), similar to the defensive parsing already used for the born timestamp in the same class. 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]
