MQTT Published BUG

[XiaoKaiRen]

ha_mqtt_ledstrip.ino Line 178

void sendState() {`
  StaticJsonBuffer<BUFFER_SIZE> jsonBuffer;

  JsonObject& root = jsonBuffer.createObject();

  JsonObject& color = root.createNestedObject("color");
  color["r"] = red;
  color["g"] = green;
  color["b"] = blue;
  root["white_value"] = white;
  
  root["state"] = (stateOn) ? on_cmd : off_cmd;
  root["brightness"] = brightness;
  root["effect"] = currentEffect;
  root["speed"] = fxspeed;
  

  char buffer[root.measureLength() + 1];
  root.printTo(buffer, sizeof(buffer));

  client.publish(light_state_topic, buffer, true);
}

if r = 255 g = 255 b = 255 can't send message

i fix Line 187

  root["w"] = white;

reduce the length of mqtt message
this can work!!

[rorist]

Hello,

Thanks for the report, is there a maximum length of the message that's been reached there ?
I don't think I have a problem setting rgb 255.

Cheers

[XiaoKaiRen]

This https://github.com/knolleary/pubsubclient/blob/54be6e87db16d7dbc60d4c4674566ec08b0115c2/src/PubSubClient.cpp

Line 366

MQTT_MAX_PACKET_SIZE = 128

boolean PubSubClient::publish(const char* topic, const uint8_t* payload, unsigned int plength, boolean retained) {
    if (connected()) {
 here --->  if (MQTT_MAX_PACKET_SIZE < 5 + 2+strlen(topic) + plength) {
            // Too long
            return false;
        }
        // Leave room in the buffer for header and variable length field
        uint16_t length = 5;
        length = writeString(topic,buffer,length);
        uint16_t i;
        for (i=0;i<plength;i++) {
            buffer[length++] = payload[i];
        }
        uint8_t header = MQTTPUBLISH;
        if (retained) {
            header |= 1;
        }
        return write(header,buffer,length-5);
    }
    return false;
}

[rorist]

Ok perfect, if you want you can do a pull request so the commit is properly credited to you ? Otherwhise I can do the change. Nice catch !