1technophile / OpenMQTTGateway

MQTT gateway for ESP8266 or ESP32 with bidirectional 433mhz/315mhz/868mhz, Infrared communications, BLE, Bluetooth, beacons detection, mi flora, mi jia, LYWSD02, LYWSD03MMC, Mi Scale, TPMS, BBQ thermometer compatibility & LoRa.
https://docs.openmqttgateway.com
GNU General Public License v3.0
3.62k stars 797 forks source link

[SYS] Log Theengs props deserilization failures #1907

Closed ssalonen closed 8 months ago

ssalonen commented 8 months ago

Now we log failing theengs json props deserialization failures.

Note: the rest of the function implementation is kept as is, with no null/data-validity checks, and NoMemory-erroring deserialization is likely to follow with a crash later in the strcmp conditionals like in #1905.

The code could be implemented more defensivly, in such a way that approriate null checks are in place before strcmp...then the code would work with "best-effort" basis in limited-memory (non-ESP32) environments. However, as pointed by the main author 1technophile in #1905, support for other boards is likely dropped in the future.

Related to #1905

Description:

Checklist:

1technophile commented 8 months ago

Thanks