Error in MqttLib/Core/QosManage.cs

stevenlovegrove / MqttDotNet

.net Implementation of the client half of the MQTT messaging protocol. MQTT is a lightweight, open specification publish and subscribe protocol cheifly developed by IBM. Details of MQTT can be found at http://mqtt.org.

MIT License

153 stars 91 forks source link

Error in MqttLib/Core/QosManage.cs #2

Open glenlivet opened 11 years ago

glenlivet commented 11 years ago

Hi,

Please check out Core/QosManage.cs Line 91. I believe that should be a MqttPubrecMessage rather than MqttPubrelMessage that is added to response. I tried to fixed it, but still didn't get it work properly. Is there anyone here who can help fix this?

ericvoid commented 9 years ago

It really seems to be wrong, since the response of a Publish with QoS=2 should be Pubrec.

stevenlovegrove commented 9 years ago

It sounds like this issue is related to #3 ?

I'll happily accept a pull request if someone has a fix and there is consensus on the correct behaviour.