commit f18674a32021a1f52d93667073ac4b86f74db15c
author Michael Scott <michael.scott@linaro.org> Thu Aug 10 16:14:03 2017 -0700
committer Jukka Rissanen <jukka.rissanen@linux.intel.com> Fri Aug 11 09:45:02 2017 +0300
tree 83ca467870b0f0eac0b19b4a41386de0478322ec
parent bd9a1548ace9e717d5103e21f7a7e02776b246f1

net: http: fix http client request "Host" header

RFC-7230 "HTTP/1.1 Message Syntax and Routing" Section 5.4
describes the "Host" header formatting.  If Zephyr user
specifies a host string as a part of the HTTP client request
structure, we end up sending an incorrect HTTP header due
to a missing "Host :" text.

Fix this by prepending "Host: " to the header data before
the user supplied host string.

Signed-off-by: Michael Scott <michael.scott@linaro.org>

subsys/net/lib/http/http_client.c

diff --git a/subsys/net/lib/http/http_client.c b/subsys/net/lib/http/http_client.c
index 1f2e80c..25f39e3 100644
--- a/subsys/net/lib/http/http_client.c
+++ b/subsys/net/lib/http/http_client.c
@@ -45,6 +45,7 @@
 /* HTTP client defines */
 #define HTTP_EOF           "\r\n\r\n"
 
+#define HTTP_HOST          "Host: "
 #define HTTP_CONTENT_TYPE  "Content-Type: "
 #define HTTP_CONT_LEN_SIZE 64
 
@@ -87,6 +88,11 @@
 	}
 
 	if (req->host) {
+		if (!net_pkt_append_all(pkt, strlen(HTTP_HOST),
+					(u8_t *)HTTP_HOST, timeout)) {
+			goto out;
+		}
+
 		if (!net_pkt_append_all(pkt, strlen(req->host),
 					(u8_t *)req->host, timeout)) {
 			goto out;