Fix meter lifecycle and MQTT/HA integration bugs #1

Merged

adebaumann merged 0 commits from refs/pull/1/head into main

2026-06-26 23:53:16 +00:00

adebaumann commented

2026-06-26 23:52:11 +00:00

(Migrated from git.baumann.gr)

Summary

Fixes a batch of meter-lifecycle and MQTT/Home Assistant integration bugs.

Detach freed PWM channels in attachMeters() — channels orphaned when the meter count shrinks or a pin changes are now detached and driven low (meter reads zero) instead of left driving PWM on the old pin.
Clear retained HA discovery for removed meters so Home Assistant drops stale entities instead of keeping them forever.
Shorten MQTT probe timeout (1500 ms → 400 ms) to limit how long the blocking probe stalls the web server per reconnect attempt.
Allow anonymous MQTT brokers — only require a host; empty credentials are passed as NULL for an anonymous connection.
Wire MQTT availability into HA discovery — LWT/status now use online/offline, and discovery includes availability_topic + payloads so HA tracks online/offline state.
Retain HA command messages (retain: true in discovery) so commanded values survive restarts.
Escape < and > in escHtml to prevent stored XSS via config fields.
Persist MQTT settings when disabling MQTT — connection fields are saved regardless of the enable checkbox.

## Summary Fixes a batch of meter-lifecycle and MQTT/Home Assistant integration bugs. - **Detach freed PWM channels** in `attachMeters()` — channels orphaned when the meter count shrinks or a pin changes are now detached and driven low (meter reads zero) instead of left driving PWM on the old pin. - **Clear retained HA discovery** for removed meters so Home Assistant drops stale entities instead of keeping them forever. - **Shorten MQTT probe timeout** (1500 ms → 400 ms) to limit how long the blocking probe stalls the web server per reconnect attempt. - **Allow anonymous MQTT brokers** — only require a host; empty credentials are passed as `NULL` for an anonymous connection. - **Wire MQTT availability into HA discovery** — LWT/status now use `online`/`offline`, and discovery includes `availability_topic` + payloads so HA tracks online/offline state. - **Retain HA command messages** (`retain: true` in discovery) so commanded values survive restarts. - **Escape `<` and `>` in `escHtml`** to prevent stored XSS via config fields. - **Persist MQTT settings when disabling MQTT** — connection fields are saved regardless of the enable checkbox.