AndrewFG
(Andrew Fiddian-Green)
April 23, 2021, 2:33pm
16
@rossko57 having looked into the code, I think your hypothesis is correct; it looks as though there are three override classes (EspMilight, Homie, HomeAssistant) that extend from the “generic” class, so my guess is that the override classes are doing stuff themselves that the “generic” class is wrongly trying to clean up for them; so I have filed a new issue; but I have not yet had time to fix the code.
opened 02:24PM - 23 Apr 21 UTC
closed 12:18PM - 08 Aug 21 UTC
bug
The MQTT AbstractBrokerHandler produces erroneous logger warnings when OH is bei… ng shutdown for the EspMilight, HomeAssistant, and Homie, discovery services. The messages are either _"Tried to unsubscribe .. but topic not registered **at all**."_ or _"Tried to unsubscribe .. but topic not registered **for listener**"._ See logs below.
One hypothesis may be that the over-riding EspMilight, HomeAssistant, and Homie classes are implementing discovery services which are (un)subscribing their respective topics (or not), and that the overidden AbstractBrokerHandler base class is not being made aware of the success or failure of this. Or something like that :)
---
**Homie**
```
2021-04-23 15:02:00.011 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homie.internal.discovery.Homie300Discovery@98ea9b from discovery topic homeassistant/# on broker mqtt:broker:g24 but topic not registered for listener. Check discovery logic!
2021-04-23 15:02:00.023 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homie.internal.discovery.Homie300Discovery@98ea9b from discovery topic milight/states/# on broker mqtt:broker:g24 but topic not registered for listener. Check discovery logic!
```
---
**HomeAssistant**
```
2021-04-23 15:02:00.043 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homeassistant.internal.discovery.HomeAssistantDiscovery@1d7aa46 from discovery topic +/+/$homie on broker mqtt:broker:g24 but topic not registered at all. Check discovery logic!
2021-04-23 15:02:00.044 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homeassistant.internal.discovery.HomeAssistantDiscovery@1d7aa46 from discovery topic milight/states/# on broker mqtt:broker:g24 but topic not registered for listener. Check discovery logic!
2021-04-23 15:02:00.047 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homeassistant.internal.discovery.HomeAssistantDiscovery@1d7aa46 from discovery topic homeassistant/# on broker mqtt:broker:g24 but topic not registered at all. Check discovery logic!
2021-04-23 15:02:00.048 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homeassistant.internal.discovery.HomeAssistantDiscovery@1d7aa46 from discovery topic +/+/$homie on broker mqtt:broker:g24 but topic not registered at all. Check discovery logic!
2021-04-23 15:02:00.050 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.homeassistant.internal.discovery.HomeAssistantDiscovery@1d7aa46 from discovery topic milight/states/# on broker mqtt:broker:g24 but topic not registered for listener. Check discovery logic!
```
---
**EspMilight**
```
2021-04-23 15:02:00.119 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.espmilighthub.internal.discovery.EspMilightHubDiscoveryService@1e70fcb from discovery topic homeassistant/# on broker mqtt:broker:g24 but topic not registered at all. Check discovery logic!
2021-04-23 15:02:00.121 [WARN ] [g.mqtt.handler.AbstractBrokerHandler] - Tried to unsubscribe org.openhab.binding.mqtt.espmilighthub.internal.discovery.EspMilightHubDiscoveryService@1e70fcb from discovery topic +/+/$homie on broker mqtt:broker:g24 but topic not registered at all. Check discovery logic!
```
2 Likes