[ 
https://issues.apache.org/jira/browse/AMQ-5101?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13934975#comment-13934975
 ] 

Dejan Bosanac commented on AMQ-5101:
------------------------------------

I applied the patch 
http://git-wip-us.apache.org/repos/asf/activemq/commit/ac23b017

However, I'm seeing quite regular failings of testResendMessageId test now

{code}Tests run: 27, Failures: 0, Errors: 1, Skipped: 1, Time elapsed: 32.601 
sec <<< FAILURE! - in org.apache.activemq.transport.mqtt.MQTTSSLTest
testResendMessageId(org.apache.activemq.transport.mqtt.MQTTSSLTest)  Time 
elapsed: 1.047 sec  <<< ERROR!
java.net.ProtocolException: Command from server contained an invalid message 
id: 4
        at 
org.fusesource.mqtt.client.CallbackConnection.completeRequest(CallbackConnection.java:723)
        at 
org.fusesource.mqtt.client.CallbackConnection.processFrame(CallbackConnection.java:762)
        at 
org.fusesource.mqtt.client.CallbackConnection.access$1500(CallbackConnection.java:51)
        at 
org.fusesource.mqtt.client.CallbackConnection$6.onTransportCommand(CallbackConnection.java:392)
        at 
org.fusesource.hawtdispatch.transport.TcpTransport.drainInbound(TcpTransport.java:709)
        at 
org.fusesource.hawtdispatch.transport.SslTransport.drainInbound(SslTransport.java:276)
        at 
org.fusesource.hawtdispatch.transport.TcpTransport$6.run(TcpTransport.java:588)
        at 
org.fusesource.hawtdispatch.internal.NioDispatchSource$3.run(NioDispatchSource.java:209)
        at 
org.fusesource.hawtdispatch.internal.SerialDispatchQueue.run(SerialDispatchQueue.java:100)
        at 
org.fusesource.hawtdispatch.internal.pool.SimpleThread.run(SimpleThread.java:77){code}

It's probably due the timeouts introduced in the test, but nevertheless it 
might be good to investigate

> MQTT SUBACK packet MUST use return code 0x80 to report failed topic 
> subscriptions
> ---------------------------------------------------------------------------------
>
>                 Key: AMQ-5101
>                 URL: https://issues.apache.org/jira/browse/AMQ-5101
>             Project: ActiveMQ
>          Issue Type: Bug
>          Components: MQTT
>    Affects Versions: 5.9.0
>            Reporter: Dhiraj Bokde
>            Assignee: Dejan Bosanac
>             Fix For: 5.10.0
>
>         Attachments: AMQ-5101.patch
>
>
> MQTT 3.1.1 spec requires that failed topic subscriptions in a SUBACK packet 
> MUST have a return code of 0x80 instead of the requested QoS. 
> A patch is included to add this behavior. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to