Fix Domoticz re-subscribe on MQTT reconnect. Regression from v13.4.0.3 (#21281)

This commit is contained in:
Theo Arends 2024-05-16 14:43:30 +02:00
parent f602470aeb
commit 6ba59385d4
3 changed files with 6 additions and 3 deletions

View File

@ -13,9 +13,10 @@ All notable changes to this project will be documented in this file.
- ESP32 compiler option from `target-align` to `no-target-align` (#21407) - ESP32 compiler option from `target-align` to `no-target-align` (#21407)
### Fixed ### Fixed
- Domoticz re-subscribe on MQTT reconnect. Regression from v13.4.0.3 (#21281)
### Removed ### Removed
- Support of old insecure fingerprint algorithm - deprecated since v8.4.0 (#21417) - Support of old insecure fingerprint algorithm. Deprecated since v8.4.0 (#21417)
## [Released] ## [Released]

View File

@ -128,6 +128,7 @@ The latter links can be used for OTA upgrades too like ``OtaUrl https://ota.tasm
- ESP32 compiler option from `target-align` to `no-target-align` [#21407](https://github.com/arendst/Tasmota/issues/21407) - ESP32 compiler option from `target-align` to `no-target-align` [#21407](https://github.com/arendst/Tasmota/issues/21407)
### Fixed ### Fixed
- Domoticz re-subscribe on MQTT reconnect. Regression from v13.4.0.3 [#21281](https://github.com/arendst/Tasmota/issues/21281)
### Removed ### Removed
- Support of old insecure fingerprint algorithm - deprecated since v8.4.0 [#21417](https://github.com/arendst/Tasmota/issues/21417) - Support of old insecure fingerprint algorithm. Deprecated since v8.4.0 [#21417](https://github.com/arendst/Tasmota/issues/21417)

View File

@ -232,7 +232,8 @@ void DomoticzMqttSubscribe(void) {
Domoticz->subscribe = false; Domoticz->subscribe = false;
MqttUnsubscribe(stopic); MqttUnsubscribe(stopic);
} }
if (!Domoticz->subscribe && any_relay) { // if (!Domoticz->subscribe && any_relay) { // Fails on MQTT server reconnect
if (any_relay) {
Domoticz->subscribe = true; Domoticz->subscribe = true;
MqttSubscribe(stopic); MqttSubscribe(stopic);
} }