RockteMQ-AI commented on issue #1438: URL: https://github.com/apache/rocketmq-clients/issues/1438#issuecomment-6314455148
**Issue Evaluation** Category: `type/bug` | Status: **Confirmed** The reported issue has been verified against the current codebase (master `a3d1de8`). **Root Cause:** `MessageView::receipt_handle()` at `rust/src/model/message.rs:470` calls `.unwrap()` on `Option<String>`. The field is populated directly from `system_properties.receipt_handle` in `from_pb_message`, which is `None` when the proxy does not set the pop checkpoint property (e.g., transaction checker messages built via `RecoverOrphanedTransactionCommand`). **Impact:** Any code path that calls `receipt_handle()` / `ack()` on a `MessageView` without a receipt handle will panic instead of returning an error. This affects the push consumer worker tasks and generic "process and ack" helpers. **Severity:** medium — causes runtime panic (process crash) in specific but reachable scenarios. **Java parity:** Java `MessageViewImpl#getReceiptHandle()` returns `""` when absent; the server rejects an ack with an empty handle gracefully. An automated fix proposal may be generated. Reply `/approve` to proceed with PR generation, `/revise` to request changes, or `/reject` to dismiss. --- *Automated evaluation by RockteMQ-AI* -- 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]
