/json/state updates with v: true not published to MQTT

[chatziko]

What happened?

When the "v": true option is passed to /json/state API, the updated values are not published to MQTT. If "v" is missing then the MQTT updates are published as expected.

This is always reproducible under both 0.15.1.beta2 and 0.14.4.

To Reproduce Bug

• Enable MQTT
• curl -X POST "http://<host>/json/state" -d '{"bri": 91, "on": true,"v":true}' -H "Content-Type: application/json"
• Observe the MQTT topics not getting updated
• curl -X POST "http://<host>/json/state" -d '{"bri": 100, "on": true}' -H "Content-Type: application/json"
• Observe the update being published to MQTT

Expected Behavior

Updates with "v": true should be published to MQTT.

Install Method

Binary from WLED.me

What version of WLED?

0.15.1.beta2

Which microcontroller/board are you seeing the problem on?

ESP32

Relevant log/trace output

Anything else?

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[DedeHai]

AFAIK WLED has not special treatment for HA. If HA uses the API incorrectly, not much we can do.

[blazoncek]

There are two aspects when MQTT message is not published: one is cooldown the other is WS only notification.
Cooldown prevents spamming interfaces with frequent updates. Will not change.
WS only notifications are intended to notify WS clients only. HA is not WS client AFAIK.

[chatziko]

Thanks for the response, after some digging intro the HA code I isolated the problem, updates are not sent when "v": true is passed to /json/state. I updated the issue to reflect this.

[blazoncek]

This is intentional. "v" is verbose response, meaning client is interested in getting the state response.

[chatziko]

This is intentional. "v" is verbose response, meaning client is interested in getting the state response.

But then what's the point of subscribing to MQTT? Whoever subscribed is also interested in the same info.

Moreover, websocket clients get the updates even when "v" is set, why are they different than MQTT clients?

[blazoncek]

Not my code and can't judge why it was implemented so. Feel free to provide a patch.