Fix handling of chunked transfer encoding (#1975)

igrr · igrr · commit 76e322f2e7c7 · 2016-04-26T15:58:12.000+08:00
diff --git a/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp b/libraries/ESP8266HTTPClient/src/ESP8266HTTPClient.cpp
@@ -624,6 +624,13 @@ int HTTPClient::writeToStream(Stream * stream)
                 break;
             }
 
+            // read trailing \r\n at the end of the chunk
+            char buf[2];
+            auto trailing_seq_len = _tcp->readBytes((uint8_t*)buf, 2);
+            if (trailing_seq_len != 2 || buf[0] != '\r' || buf[1] != '\n') {
+                return returnError(HTTPC_ERROR_READ_TIMEOUT);
+            }
+
             delay(0);
         }
     } else {

Original file line number	Diff line number	Diff line change
`@@ -624,6 +624,13 @@ int HTTPClient::writeToStream(Stream * stream)`
`624`	`624`	`break;`
`625`	`625`	`}`
`626`	`626`
	`627`	`+ // read trailing \r\n at the end of the chunk`
	`628`	`+ char buf[2];`
	`629`	`+ auto trailing_seq_len = _tcp->readBytes((uint8_t*)buf, 2);`
	`630`	`+ if (trailing_seq_len != 2 \|\| buf[0] != '\r' \|\| buf[1] != '\n') {`
	`631`	`+ return returnError(HTTPC_ERROR_READ_TIMEOUT);`
	`632`	`+ }`
	`633`	`+`
`627`	`634`	`delay(0);`
`628`	`635`	`}`
`629`	`636`	`} else {`