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

Pin Paho to <2.0.0 #164

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

digitalgravy
Copy link

Paho has recently been updated to 2.0.0 and has broken the current API. Some details can be found in this issue: eclipse-paho/paho.mqtt.python#814

journalctl -u shutters.service -e will show this output if Paho 2.0.0 is installed and the instruction on Pi-Somfy's readme are followed:

Exception in thread MQTT:
Traceback (most recent call last):
  File "/usr/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
 * Serving Flask app 'WebServer'
  File "/home/stue/Pi-Somfy/mymqtt.py", line 149, in run
 * Debug mode: off
    self.t = paho.Client(mqtt_client.CallbackAPIVersion.VERSION1, self.config.MQTT_ClientID)
NameError: name 'mqtt_client' is not defined

This change pins Paho to < 2.0.0 thus avoiding this error.

If anyone would like to update Pi-Somfy to use Paho >= 2.0.0 then there are some instructions to follow: https://github.com/eclipse/paho.mqtt.python/blob/master/docs/migrations.rst

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 this pull request may close these issues.

1 participant