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

MQTT plugin freezes PlotJuggler on Linux. #1008

Closed
Valle125 opened this issue Sep 21, 2024 · 0 comments · Fixed by #1009
Closed

MQTT plugin freezes PlotJuggler on Linux. #1008

Valle125 opened this issue Sep 21, 2024 · 0 comments · Fixed by #1009

Comments

@Valle125
Copy link
Contributor

The MQTT plugin causes PlotJuggler 3.9.1 to freeze on Linux (Ubuntu 22) if the connect button is hit repeatedly. If this happens PlotJuggler doesn't react to any input. The program must be terminated e.g. by using the task manager.

Screenshot from 2024-09-21 20-24-19

facontidavide pushed a commit that referenced this issue Nov 10, 2024
* mqtt_client: fix bug where program flow gets stuck when calling mosquitto_connect_bind_v5.

The connection procedure starts with a fresh mosquitto instance each time. The instance will be destroyed on disconnect.
All mosquitto return codes are now checked during client configuration.

* mqtt_client: add debug messages and improve QMessageBox messages.

* mqtt_client: add mosquitto thread, fix #1006.

Threaded mode of libmosquitto may not be supported on windows (libmosquitto < 2.1). At some point libmosquitto has removed threaded support (mosquitto_loop_start doesn't work and returns MOSQ_ERR_NOT_SUPPORTED). Threaded support will be added again in the future, see eclipse-mosquitto/mosquitto#2707. This commit introduces a thread dedicated to mosquitto to solve the problem.

* mqtt_client: fix typo.

---------

Co-authored-by: Valentin Platzgummer <valentin.platzgummer@tuwien.ac.at>
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

Successfully merging a pull request may close this issue.

1 participant