Skip to content
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

Support MQTT3.1 #251

Open
stevenhartley opened this issue Nov 22, 2024 · 4 comments
Open

Support MQTT3.1 #251

stevenhartley opened this issue Nov 22, 2024 · 4 comments
Assignees
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@stevenhartley
Copy link
Contributor

For MQTT 3.1, the UMessage is not split up putting the attributes in the MQTT header and the payload in the MQTT payload but the entire UMessage is stuffed inside of the MQTT3.1 payload. This PR is to update the MQTT transport spec to support 3.1

@ValMobBIllich
Copy link

At first glance yeah I think its fine to say that with this change we simply support MQTT3.1. Ill think about this a little bit more though. We made a kind of similar change in a previous PR where we allow messages to not have subscription identifiers (even though those are required per the mqtt5 spec) because the azure brokers dont support them (even though they claim to be mqtt5 compliant...).

The standards seem to be a bit murky and the requirements optional in the MQTT world...

@stevenhartley stevenhartley added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 22, 2024
@sophokles73
Copy link
Contributor

IMHO we could/should use the CloudEvent Protobuf Format for this purpose as defined in CloudEvents and updated in #248 ...

@PLeVasseur
Copy link
Contributor

@ValMobBIllich -- did you have feedback on @sophokles73's proposal?

@sophokles73
Copy link
Contributor

BTW up-rust already contains functionality for mapping UMessages <-> CloudEvents using Protobuf Format.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants