[
https://issues.apache.org/jira/browse/AMQ-5975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14900738#comment-14900738
]
Timothy Bish commented on AMQ-5975:
-----------------------------------
This is a client side issue, the specification for STOMP defines that the
values in the CONNECT and CONNECTED do not need to be escaped:
{quote}
Escaping is needed to allow header keys and values to contain those frame
header delimiting octets as values. The CONNECT and CONNECTED frames do not
escape the carriage return, line feed or colon octets in order to remain
backward compatible with STOMP 1.0.
{quote}
> invalid header errror
> ---------------------
>
> Key: AMQ-5975
> URL: https://issues.apache.org/jira/browse/AMQ-5975
> Project: ActiveMQ
> Issue Type: Bug
> Components: stomp
> Affects Versions: 5.11.1
> Reporter: Abhi
> Labels: stomp, v1.1, v1.2
>
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: encoding CONNECT frame
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H passcode:
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H
> accept-version:1.0,1.1,1.2
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H host:bismuth31.nyc
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H login:
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: sent 73 bytes
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: received 123 bytes
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: decoding CONNECTED frame
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H server:ActiveMQ/5.11.1
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H heart-beat:0,0
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H
> session:ID:bismuth31.nyc-47753-1442302511794-1:3
> # 2015/09/15-03:35:13 stomp_producer.pl[29263.0]: H version:1.2
> [20150915 03:35:13.507 Net::Stomp::connect():332 WARN] Error while connecting
> to the message broker: invalid header:
> session:ID:bismuth31.nyc-47753-1442302511794-1:3
> This is happening because session header contains ':' in its value which
> fails the check when using STOMPv1.1 protocol.
> ActiveMQ generates a default client id if one is not provided at the start
> and uses that to set session-id. But the session header cannot contain ':'.
> (https://github.com/apache/activemq/blob/138e52b08c2f49b730817932a6e63f2a135854f1/activemq-client/src/main/java/org/apache/activemq/util/IdGenerator.java
> and
> https://github.com/apache/activemq/blob/87fd0a9e054017254c3857b245ca6fb9047ccc4f/activemq-stomp/src/main/java/org/apache/activemq/transport/stomp/ProtocolConverter.java#L797)
> We can use generateSanitizedId() here to avoid this
> issue(https://github.com/apache/activemq/blob/87fd0a9e054017254c3857b245ca6fb9047ccc4f/activemq-stomp/src/main/java/org/apache/activemq/transport/stomp/ProtocolConverter.java#L797)
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
