MQTT receive command topic not working?

[SpaceTeddy]

dear proddy,

sry to issue a topic again, but since 1.2.x I've got trouble to "sent" MQTT messages to the ESP8266. receiving messages is not an issue at all.
With V. 1.1.x is was working well. As I see in the changelog, you have modified the MQTT client, therefore I would like to kindly as you if I have to do something different in V1.2.x to V1.1.x.

My test setup:
ESP8266 <-> mosquitto (rPi) <-> MQTT.fx (Mac)
User and PW is set on ESP8266 & MQTT.fx

any idea?

many thanks

chris

[SpaceTeddy]

update:
ok, if I activate the debug output in void MyESP::_mqttOnMessage
myDebug_P(PSTR("[MQTT] Received %s => %s"), topic, message);
I can clearly see that the thermostat command messages are received by the ESP, but the system does not change the mode or temperature.
Strange is, that I receive also a lot of heating_active and tapwater_active messages, but I do not send these messages 🧐
home/boiler/tapwater_active => 0
home/boiler/heating_active => 0`

BTW: I don't use HA

[proddy]

Yes, I did completely re-write all the mqtt, wifi and ota logic during the holidays and probably rushed it and under the influence of alcohol slightly! One change was to always use QoS 1. But that shouldn't make any difference. I'll take a look again The MQTT messages for heating active and tapwater active looks like a bug, or perhaps a message was sent to MQTT with the retain flag set. I'll look into that too.

…

On Thu, Jan 3, 2019 at 9:57 AM SpaceTeddy ***@***.***> wrote: update: ok, if I activate the debug output in void MyESP::_mqttOnMessage myDebug_P(PSTR("[MQTT] Received %s => %s"), topic, message); I can clearly see that the thermostat command messages are received by the ESP, but the system does not change the mode or temperature. Strange is, that I receive also a lot of heating_active and tapwater_active messages, but I do not send there messages 🧐 home/boiler/tapwater_active => 0 home/boiler/heating_active => 0` [image: image] <https://user-images.githubusercontent.com/9148964/50629391-a0b23a00-0f3c-11e9-8e90-d7fe2c9978ba.png> BTW: I don't use HA — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#27 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABLHePXvdLy843RwBF9fDuZyN25Jp7T0ks5u_cYXgaJpZM4Zns79> .

[SpaceTeddy]

well,
strcmp of the command topic is the problem, because the received topic value is home/boiler/thermostat_cmd_mode and the compare is on TOPIC_THERMOSTAT_CMD_MODE only.
so, i guess that home/boiler/ must be cut first, or compare to ...(dirty hack)...
if (strcmp(topic, "home/boiler/"TOPIC_THERMOSTAT_CMD_MODE) == 0)

or like V1.1.0
#define MQTT_BOILER MQTT_BASE APP_HOSTNAME "/"
...
if (strcmp(topic, MQTT_BOILER TOPIC_THERMOSTAT_CMD_MODE) == 0)

It is working then...
[055277] [MQTT] Received home/boiler/thermostat_cmd_mode => manual
MQTT topic event: home/boiler/thermostat_cmd_mode
MQTT message: manual
MQTT topic: thermostat mode value manual
Setting thermostat mode to 1

For tapwater_active and heating_active I deactivated for testing
//myESP.mqttSubscribe(TOPIC_BOILER_TAPWATER_ACTIVE);
//myESP.mqttSubscribe(TOPIC_BOILER_HEATING_ACTIVE);
in void MQTTcallback(unsigned int type, const char * topic, const char * message)

I should also drink something ... because I really like your project. 👍

[proddy]

yes, I had already fixed the subscribe problem. I'll fix the CMD_MODE too and do a new build soon

[SpaceTeddy]

works in 1.2.3