Merge pull request #1125 from MaOreoreorui/master

[ISSUE #1124] fix: panic in SendResult.String
apache · Mar 6, 2024 · f4875c1 · f4875c1
2 parents af567d2 + 7a89bba
commit f4875c1
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/primitive/result.go b/primitive/result.go
@@ -58,8 +58,12 @@ func NewSendResult() *SendResult {
 
 // SendResult send message result to string(detail result)
 func (result *SendResult) String() string {
+	mq := "nil"
+	if result.MessageQueue != nil {
+		mq = result.MessageQueue.String()
+	}
 	return fmt.Sprintf("SendResult [sendStatus=%d, msgIds=%s, offsetMsgId=%s, queueOffset=%d, messageQueue=%s]",
-		result.Status, result.MsgID, result.OffsetMsgID, result.QueueOffset, result.MessageQueue.String())
+		result.Status, result.MsgID, result.OffsetMsgID, result.QueueOffset, mq)
 }
 
 // TransactionSendResult RocketMQ send result