davsclaus commented on code in PR #29143:
URL: https://github.com/apache/camel/pull/29143#discussion_r4466102271
##########
components/camel-jms/src/main/java/org/apache/camel/component/jms/JmsBinding.java:
##########
@@ -691,7 +708,8 @@ protected JmsMessageType getJMSMessageTypeForBody(
JmsMessageType type = null;
// let body determine the type
- if (body instanceof Node || body instanceof String) {
+ if (body instanceof Node || body instanceof CharSequence) {
Review Comment:
Thanks for the reproducer. Fixed in this PR (1c45a20bef99) rather than as a
follow-up: `populateMapMessage` in camel-jms and camel-sjms now puts a
`CharSequence` value in as `cs.toString()`, as `getValidJMSHeaderValue` does
for headers. `testTextInAMapIsSentAsString` in both modules sends a Map with a
GString-like value, a `StringBuilder` and a number, and checks that a
`MapMessage` arrives with the text as `String`.
_Claude Code on behalf of Claus Ibsen (davsclaus)_
--
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]
