Spurious "SEND OK" in communication about esp-at HOT 10 CLOSED

Thanks for your reporting, we will revise it in the future release.

from esp-at.

This makes our current code unstable - since we have no idea when/where this pops up inside the payload. So for us it is fairly important and it is a serious bug. Are you not going to look into it?

from esp-at.

Hi gokr, we are trying to duplicate it and debug it now. I'll let you know if there is any update.

from esp-at.

Great! I can see if I can get more info from the guy on our side that has been debugging it.

from esp-at.

Just verified - when it shows up it seems to always be first in the payload. But it does not affect the size parameter (as expected) so the payload then ends up "too long" so to speak.

from esp-at.

I have the same or a very similar problem.
Sending two request directly after another, I get:

0,CONNECT

+IPD,0,306:1,CONNECT

+IPD,1,285:GET /value HTTP/1.1
Host: 192.168.4.1:333
User-Agent: Mozilla/5.0 (Android 5.1.1; Tablet; rv:53.0) Gecko/53.0 Firefox/53.0
Accept: */*
Accept-Language: de,de-DE;q=0.8,en-US;q=0.5,en;q=0.3
Accept-Encoding: gzip, deflate
Referer: http://192.168.4.1:333/
Connection: keep-alive

PUT /unblock HTTP/1.1
Host: 192.168.4.1:333
User-Agent: Mozilla/5.0 (Android 5.1.1; Tablet; rv:53.0) Gecko/53.0 Firefox/53.0
Accept: */*
Accept-Language: de,de-DE;q=0.8,en-US;q=0.5,en;q=
Accept-Encoding: gzip, deflate
Referer: http://192.168.4.1:333/
Connection: keep-alive
Content-Length: 0

In the file esp-idf/components/driver/uart.c the function uart_tx_all uses a mutex, but probably this function is called twice: first for the IPD header, then for the IPD data, so other messages can be placed between header and data.
As the calling code probably is in components/at/lib/libat_core.a, I cannot analyze the problem further without access to the library source code.

from esp-at.

Hi @hs1956,we have fixed it, but have not released it on github.

from esp-at.

Ok, good! Hoping for a release then :)

from esp-at.

@xcguang Any reason we can't get this fix in our hands? My customer is waiting for this and they have a deadline to meet.

from esp-at.

Hi @gokr , we have updated it, please have a try.

from esp-at.