client->state not set in MQTT_STATE_WAIT_TIMEOUT state

espressif / esp-mqtt

ESP32 mqtt component

Apache License 2.0

610 stars 257 forks source link

client->state not set in MQTT_STATE_WAIT_TIMEOUT state #89

Closed leewo0 closed 5 years ago

leewo0 commented 5 years ago

If auto_reconnect is disabled, client->state won't be setted in MQTT_STATE_WAIT_TIMEOUT state, which results in an error if esp_mqtt_client_start() is called.

david-cermak commented 5 years ago

Thank you @leewo0 for this useful bugfix, indeed client->state must be updated with client->run. Can you please rebase your changes? (again on top of idf branch)