server icon indicating copy to clipboard operation
server copied to clipboard

Support MQTT 5 protocol

Open mary-ng5519 opened this issue 2 years ago • 5 comments

Folks, we love your work, please help support MQTT 5 protocol, Cheers,

mary-ng5519 avatar Apr 09 '22 19:04 mary-ng5519

For the reference this has a very good implementation of MQTT V5.

saeidakbari avatar Apr 19 '22 17:04 saeidakbari

Support MQTT 5 protocol And Cluster. May refer to https://github.com/wind-c/comqtt .

wind-c avatar Sep 10 '22 13:09 wind-c

Support MQTT 5 protocol And Cluster. May refer to https://github.com/wind-c/comqtt .

would be great for us users if a way could be found to have those updates in comqtt merged back here to this project

torntrousers avatar Sep 11 '22 05:09 torntrousers

@torntrousers I will definitely be having a deeper look at @wind-c 's implementation of some of these features. I am very impressed what they have accomplished.

For mochi-co/mqtt it's important to consider the purpose of the library - which was (originally) to provide a minimalist, high performance embeddable library. We've aimed for the smallest, simplest amount of code possible in order to achieve these goals, so when it comes to features like clustering, there's a lot to consider.

I do think it would be nice to add these features, but they need to be balanced against maintainability and performance - and there's a few areas nin the existing codebase we need to improve first (like #101) :)

mochi-co avatar Sep 11 '22 07:09 mochi-co

For sure, all good for me and thats why we use your project.

torntrousers avatar Sep 11 '22 08:09 torntrousers

Resolved in v2.0.0

mochi-co avatar Dec 10 '22 22:12 mochi-co