TheNeikos / cloudmqtt

A simple and straightforward to use MQTT client/server
Other
5 stars 3 forks source link

[parse] adding short disconnect msg support #292

Closed dscso closed 1 month ago

dscso commented 1 month ago

291

dscso commented 1 month ago

Idk the second commit of the PR is much less important, but in a very data limited environment this could make sense if you are reopening the MQTT connection very often

TheNeikos commented 1 month ago

Ah good catch!

There is similar logic for this like this in the repo:

https://github.com/TheNeikos/cloudmqtt/blob/fdcfb587d4ce0ea6ff289de66ab1169554758871/mqtt-format/src/v5/packets/puback.rs#L58-L68

Do you think you could adapt it to this?

dscso commented 1 month ago

true, looks better :). Done

TheNeikos commented 1 month ago

Thank you!