RockteMQ-AI commented on issue #10688:
URL: https://github.com/apache/rocketmq/issues/10688#issuecomment-5114874986
**Issue Evaluation**
Category: `type/bug` | Status: **Confirmed**
The reported issue has been verified against the current codebase.
**Root Cause:** In `MessageQueueSelector.buildWrite()` (proxy module), the
parsing of `orderTopicConf` entries assumes every fragment contains a valid
`brokerName:queueNum` format. Specifically:
- `item[1]` is accessed without checking if `split(":")` produced enough
parts
- `Integer.parseInt(item[1])` is called without catching
`NumberFormatException`
A single malformed entry (e.g., `broker-a` or `broker-a:not-a-number`) will
crash the entire write queue selection path.
**Impact:** Proxy write path for ordered topics. One bad route fragment
breaks queue selection for all producers writing to that ordered topic.
**Severity:** medium — defensive coding gap; triggered by corrupted route
data rather than normal operation.
An automated fix proposal will be generated. Reply `/approve` to proceed
with PR generation.
---
*Automated evaluation by github-manager-bot*
--
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]
