espressif / esp-mqtt

ESP32 mqtt component
Apache License 2.0
603 stars 255 forks source link

MQTT Client: Check for connection before sending disconnect message (IDFGH-1339) #118

Closed gregjesl closed 5 years ago

gregjesl commented 5 years ago

Closes https://github.com/espressif/esp-idf/issues/3619

david-cermak commented 5 years ago

Hi @gregjesl ,

Thanks for posting this fix. LGTM. Could you please change the target branch to master? (branch idf has been recently merged to master, which is the default branch)

gregjesl commented 5 years ago

@david-cermak I just made the switch.

david-cermak commented 5 years ago

Cherry-picked as https://github.com/espressif/esp-mqtt/commit/7223302deb3ffefe60cc645c6d792fd5e4d6259c, thanks for your help!