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

mosquittopp with MQTT v5 RPC Response Topics #2782

Open
sdettmer opened this issue Apr 4, 2023 · 2 comments
Open

mosquittopp with MQTT v5 RPC Response Topics #2782

sdettmer opened this issue Apr 4, 2023 · 2 comments

Comments

@sdettmer
Copy link

sdettmer commented Apr 4, 2023

Hi,

I'm trying to use MQTT v5 RPC Response Topics with Mosquitto. There is a mosquitto_rr tool supporting it, so I assume the mosquitto lib suopport it (via mosquitto_subscribe_v5 / mosquitto_subscribe_multiple), however I find no way to call it via mosquittopp. I cannot even implement in a child class, because unfortunately the struct mosquitto *m_mosq; is private:.

What do I do wrongly?

In case it is not possibly, I suggest to:

  1. add the missing functionality to mosquittopp
  2. verify other missing functionality (I guess, all properties-aware functionality could be missing)
  3. make struct mosquitto *m_mosq; to be protected: or add a protected: accessor (getter).

I think I'm missing a point, because missing properties would make mosquittopp not MQTT v5 compliant I think, so maybe it is not maintained (?) and nobody is using it? Also the documentation https://mosquitto.org/api/files/cpp/mosquittopp-h.html is quite brief and contains DEPRECATED. Is mosquittopp deprecated, and if so, by what?

Would it be welcome to propose patches?

Regards,
Steffen

@ralight
Copy link
Contributor

ralight commented Apr 6, 2023

mosquittopp was deprecated because I saw approximately zero use, was extra burden to maintain, was only an extremely thin wrapper around the C library, and the Paho C++ library provided a better C++ experience. After suggesting it was deprecated, there was sufficient interest to reverse the decision - the API documentation is out of date by a reasonable while in that regard.

The C++ wrapper has been updated in the develop branch - I'd be grateful if you gave it a try and reported back if it is all working for you.

@vimpostor
Copy link

I'd be grateful if you gave it a try and reported back if it is all working for you.

I tested the C++ bindings and they seem to be working fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants