-
Notifications
You must be signed in to change notification settings - Fork 175
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
mqtt5 #123
Comments
Nothing immediate but this does sound like an update the client will need, I'll have a look at the work involved and update this issue. |
Any update on this? Time frame maybe please. |
Is there a time frame to bring this enhancement? Is there any dev version which I could test? |
I am starting to look at this but there is no time frame as of yet |
OK, I think what I'll do is implement MQTT 5 in chunks rather than all at once, this will lower the time frame to a working albeit not fully compliant MQTT 5 client. beside the addition of message properties and the new message structure what other functionality do you need for a first release? |
oh grat, that sounds really good. |
I checked the needed featues for my project. |
Yep OK, I'll add the enhanced auth method, still can't give you a timeframe here but at least I've started and we have an end goal. If you want to keep an eye on things the updates will be in the mqtt5 branch |
OK, forget the branch idea, there's going to be a lot of change so for now I've made this its own client here. This allows me to develop more freely without worrying about breaking existing functionality and littering the code with class renames and 'if version ===' statements. In the long run it may stay like this with maybe the addition of a common support package for both clients. Use this issue to track progress and raise concerns/suggestions etc. |
The mqtt5_client is now published at version 1.0.0, closing this issue. |
Is there any plans for implementing mqtt5, my use case will require user properties, which is added on mqtt5, i wonder if there is a plan for it, or is it already on dev repo which i can test?
The text was updated successfully, but these errors were encountered: