-
Notifications
You must be signed in to change notification settings - Fork 2.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
MQTT 5 support for bridges #1292
Comments
Yes! I didn't want to delay 1.6 to put it in though. |
Great news. I took a look on the code, and it look like I just have to patch the existing code that load the bridge configuration (see below) for adding the support of the v5 parameter. see conf.c I hope that after this the bridge will act as a standard mqtt client supporting mqtt v5. I would like to test that feature before an official release. Thks |
Possibly! I feel as though there should be more to it than that though. I've not got time to look at this right at the moment, so feel free to experiment and report back if you want. Something you could try would be to modify the tests in test/broker/06* to use the new mqttv5 bridge option. That would give us some basic operation at least. Then other changes like modifying subscription options to support the "no local" and "propagate retain" features in MQTTv5 would be needed. At the moment these options are produced in a non standardised way. |
Is it possible to add the support of MQTT 5 with bridges ?
The text was updated successfully, but these errors were encountered: