Skip to content

Conversation

emelianov
Copy link
Contributor

Implement early exit as connection established or data already sent.
(In my tests previous implementation was exiting only on timeout expired)

Implement early exit as connection established or data already sent.
(Previous implementation was exiting only on timeout expired)
@d-a-v
Copy link
Collaborator

d-a-v commented Jun 24, 2019

reference for reviewers: d-a-v/W5500lwIP#6 (comment)

@d-a-v d-a-v requested review from earlephilhower and devyte June 24, 2019 13:23
@d-a-v d-a-v merged commit 4bfa2ae into esp8266:master Jul 5, 2019
@emelianov emelianov mentioned this pull request Jul 8, 2019
@ascillato
Copy link
Contributor

Hi,

@emelianov @d-a-v @devyte @earlephilhower

This PR makes the pubsubclient library to hung (device unresponsive) for 5 seconds if the MQTT broker is down and it is trying to reconnect.

Reverting just this commit makes the STAGE core to be fast again as core 2.5.2 while using MQTT.

I have just made the PR #6440 in order to fix this hung issue. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants