RockteMQ-AI commented on issue #10939: URL: https://github.com/apache/rocketmq/issues/10939#issuecomment-5394935209
**Issue Evaluation** Category: `bug` | Status: **Confirmed** Clear null-safety issue. `AbstractRemotingActivity.request` accesses `request.getExtFields().get(...)` without null-checking the ext-fields map first, converting a valid protocol-level scenario into an NPE (500 Internal Server Error). **Root Cause:** Missing null guard on `getExtFields()` before field lookup. **Impact:** Any Remoting request with null ext fields (valid per protocol) triggers an NPE in the Proxy, resulting in 500 responses instead of proper protocol-level error handling. **Severity:** Medium The proposed fix (guard ext-fields lookup, add regressions for both normal forwarding and SendMessageV2 variant) is well-scoped. --- *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]
