search

G-Two / smoke-x-receiver

An ESP32+LoRa application to bridge a ThermoWorks Smoke X remote thermometer to Home Assistant via MQTT
MIT License
14 stars 5 forks source link

Increased MQTT buffer size for Smoke X4 #8

Closed fastbytes closed 1 year ago

fastbytes commented 1 year ago

Increased MQTT message buffer size from 512->1024 to prevent truncation when using Smoke X4.

Before (truncated to 512 characters):

{"probe_1_attached":"ON","probe_1_alarm":"OFF","probe_1_temp":68.5,"probe_1_max":200,"probe_1_min":32,"billows_target":"offline","probe_2_attached":"ON","probe_2_alarm":"OFF","probe_2_temp":69.5,"probe_2_max":200,"probe_2_min":32,"billows_target":"offline","probe_3_attached":"ON","probe_3_alarm":"OFF","probe_3_temp":68.8,"probe_3_max":160,"probe_3_min":32,"billows_target":"offline","probe_4_attached":"OFF","probe_4_alarm":"offline","probe_4_temp":"offline","probe_4_max":"offline","probe_4_min":"offline",

After (533 characters):

{"probe_1_attached":"ON","probe_1_alarm":"OFF","probe_1_temp":68.4,"probe_1_max":200,"probe_1_min":32,"billows_target":"offline","probe_2_attached":"ON","probe_2_alarm":"OFF","probe_2_temp":69.5,"probe_2_max":200,"probe_2_min":32,"billows_target":"offline","probe_3_attached":"ON","probe_3_alarm":"OFF","probe_3_temp":68.8,"probe_3_max":160,"probe_3_min":32,"billows_target":"offline","probe_4_attached":"OFF","probe_4_alarm":"offline","probe_4_temp":"offline","probe_4_max":"offline","probe_4_min":"offline","billows_attached":"ON"}
G-Two commented 1 year ago

Thanks for finding/fixing the bug. There might be other X4 bugs around, since I wrote this with only an X2 in hand.