-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
Upgrade MQTTNet to latest version, add support for .NET 5 #163
base: master
Are you sure you want to change the base?
Upgrade MQTTNet to latest version, add support for .NET 5 #163
Conversation
Some WIP on our side: we're still testing it with a live MQTT queue, so not 100% ready to merge yet. |
@keesschollaart81 can this be merged and possibly updated for NET6 and latest MQTTNet package? |
Will look into this next weekend. LGTM! |
Sorry to say that we didn't get this working on our end; I would recommend against merging as-is. Skipping .NET 5 and going straight to .NET 6 will most likely give you a much better experience for upgrading. |
Good that you're report this! What was the issue / should I look out for? |
Honestly; actually receiving a message from a test queue. I don't remember exactly because it's been too long ago. The different modes of running it isolated / non-isolated ended up biting us in the rear as far as I remember. |
@duizendnegen I'm most interested in running NET6 Isolated - actually about to try it - as Microsoft claims it's the future of Functions. Do you recall if isolated was the problem? |
I tried to start porting to NET6/Isolated but as you probably know configuration & bindings are way different. I wasn't able to complete a working version of this library yet. |
I had a similar issue; configurations / bindings being way different. |
We are running .NET 5 with our Azure Functions; this PR updates this project to support that.