quartiq / minimq

Minimal no_std MQTT v5.0 client implementation
MIT License
143 stars 16 forks source link

`UserProperty` is serialized backwards #122

Closed ryan-summers closed 1 year ago

ryan-summers commented 1 year ago

The UserProperty is incorrectly serializing the value followed by the key, which means that the order is unintentionally swapped.

This is likely a holdover from when we used to invert serialization to simplify lengths.