arnoudja commented on PR #1851:
URL: https://github.com/apache/activemq/pull/1851#issuecomment-4147886696
As a result to your remark "Openwire messages should arrive on the broker
and be serialized already into bytes":
As far as I can tell, the ActiveMQTextMessage instance in this path is
created in this method in JMSMappingInboundTransformer.java:
public static ActiveMQTextMessage createTextMessage(String text) {
ActiveMQTextMessage message = new ActiveMQTextMessage();
try {
message.setText(text);
} catch (MessageNotWriteableException ex) {}
return message;
}
As you can see, setText is called which will fill the text field, not the
content field. Adding a call to setContent here might solve the problem but
this is probably not the best place to do so.
Also, for the STOMP part, does
https://github.com/apache/activemq/commit/57264bf8dc9970d7d808cf24216b6185ceb644e2
address this? That wouldn't, however, explain why we still had the problem
before this ActiveMQMessageText change.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information, visit: https://activemq.apache.org/contact
