AsyncClient::add() never returns

[drejkus]

Hi,
I have issue with AsyncClient::add() - it never returns, when adding only 2 bytes.
I'm actually trying to modify https://github.com/marvinroger/async-mqtt-client to work with ESP32, but it fails right at the beginning in the onConnect callback where it tries to assemeble the packet.

Please do you have any clue what goes wrong or where to look?

[me-no-dev]

Hmm... that is rather strange... need to investigate

[drejkus]

Just adding simple test file.

AsyncTest.zip

[copercini]

I'm hitting the exactly same issue, one of the problems is in dns resolution, AsyncMqttClient doesn't have a dns_found_callback function to continue after AsyncTCP return an IP here: https://github.com/me-no-dev/AsyncTCP/blob/master/src/AsyncTCP.cpp#L555

with IP directly, seems to be something between _tcp_write and _tcp_write_api because none of my debug messages in _tcp_write_api are printed, and the last one is in _tcp_write

[me-no-dev]

Aaaahhhh... onConnect runs on the network thread... let me try to figure this out...

[me-no-dev]

guys try the latest commit :)

[drejkus]

Brilliant! Works for me.

@copercini - I hit the DNS issue aswell, but I belive that the problem is in AsyncMqttClient where it does not return AsyncTCP connect() result in it's own connect(). If it did we could simply issue a reconnect. But that is another story

[me-no-dev]

dns issue should also be fixed ;)

[copercini]

Awesome man!!!

Both problems was solved here, thanks!

[me-no-dev]

:) closing then :P