search

camueller / SmartApplianceEnabler

This gateway software turns existing devices into smart appliances.
GNU General Public License v2.0
128 stars 40 forks source link

Keine Aktualisierung im Sunny-Portal nach MQTT-Broker Down/Up #451

Closed J0EK3R closed 1 year ago

J0EK3R commented 1 year ago

Hallo Axel - hab erfolglos versucht, den Bug #447 nochmal zu öffnen, deshalb ein neuer:

Ich habe mir die Änderung mit dem Zwischenspeichern der Subscriptions in der map messageHandlerForSubscribedTopic nochmal in Ruhe angeschaut...

Ich denke, Put und auch Remove von Topic und Handler sollten nicht von connect() abhängig sein, also auch im disconnect-Zustand in/aus der map gepackt/gelöscht werden können. Deshalb sollten die Aufrufe also aus dem if(connect())-Block raus.

Dann ist mir noch was aufgefallen: kann es sein, dass das "messageHandlerForSubscribedTopic.put" eher in die private Methode subscribe gehört? Also die, die auch client.subscribe aufruft?!

Beste Grüße Jochen

camueller commented 1 year ago

Diskussion wird in https://github.com/camueller/SmartApplianceEnabler/issues/447 fortgesetzt.