search

ekstrand / ESP8266wifi

ESP8266 Arduino library with built in reconnect functionality
MIT License
451 stars 235 forks source link

Update ESP8266wifi.cpp #5

Closed Creamers158 closed 9 years ago

Creamers158 commented 9 years ago

Typo.

I found out that posting to thingspeak doesnt work probably because you force CIPSTART to id 4.

SERVER value and cipstart and cipmux are hardcoded and not dynamic. Is there a way to set it to single mode to see if it works against thingspeak?

const char CIPSTART[] PROGMEM = "AT+CIPSTART=4,\"";" to const char CIPSTART[] PROGMEM = "AT+CIPSTART=\"";

and const char CIPMUX_1[] PROGMEM = "AT+CIPMUX=1"; to const char CIPMUX_1[] PROGMEM = "AT+CIPMUX=0";

and defining SERVER to 'nothing' to CIPSEND doesnt get forced to send a defined id. Seems like thingspeak doesnt allow multiple connections or there is something else wrong.

[System Ready, Vendor:www.ai-thinker.com] AT+CIPSTART="TCP","184.106.153.149",80

OK Linked AT+CIPSEND=4,13

type error Ping ping..

Error

ekstrand commented 9 years ago

Hi, Actually 4 is not the type, its the connection ID, I always use 4 as the ID for server connections. The type is either TCP or UDP. See docs at https://github.com/espressif/esp8266_at/wiki/CIPSTART https://github.com/espressif/esp8266_at/wiki/CIPSTART

AT+CIPSTART=id,type,addr,port id: 0-4, id of connection type: string, “TCP” or “UDP” addr: string, remote ip port: string, remote port

You do know that the library has not built in support for http, it’s pure TCP or UDP only today. Regards Jonas

25 apr 2015 kl. 12:31 skrev Creamers158 notifications@github.com:

Typo.

I found out that posting to thingspeak doesnt work because you force CIPSTART to type 4. Everything works, but somehow you for CIPSEND also to 4. Any suggestions or examples?

[System Ready, Vendor:www.ai-thinker.com] AT+CIPSTART="TCP","184.106.153.149",80

OK Linked AT+CIPSEND=4,13

type error Ping ping..

Error

You can view, comment on, or merge this pull request online at:

https://github.com/ekstrand/ESP8266wifi/pull/5 https://github.com/ekstrand/ESP8266wifi/pull/5 Commit Summary

Update ESP8266wifi.cpp File Changes

M ESP8266wifi.cpp https://github.com/ekstrand/ESP8266wifi/pull/5/files#diff-0 (4) Patch Links:

https://github.com/ekstrand/ESP8266wifi/pull/5.patch https://github.com/ekstrand/ESP8266wifi/pull/5.patch https://github.com/ekstrand/ESP8266wifi/pull/5.diff https://github.com/ekstrand/ESP8266wifi/pull/5.diff — Reply to this email directly or view it on GitHub https://github.com/ekstrand/ESP8266wifi/pull/5.

Creamers158 commented 9 years ago

You are right, tried to change all connections to a single connection. That works but the HTTP GET requests are not accepted by the server. Also the char buffer lenghts are not correct when sending.

ekstrand commented 9 years ago

You can change the char buffer length so that it suits your needs. /Jonas

26 apr 2015 kl. 13:25 skrev Creamers158 notifications@github.com:

You are right, tried to change all connections to a single connection. That works but the HTTP GET requests are not accepted by the server. Also the char buffer lenghts are not correct when sending.

— Reply to this email directly or view it on GitHub.