michaelh/zephyr
1
0
Fork 0
Code Releases Activity

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

Browse source 
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>
This commit is contained in:
Michael Scott 2017-08-10 16:14:03 -07:00 • committed by Jukka Rissanen
commit f18674a320
1 changed files with 6 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

6
subsys/net/lib/http/http_client.c
View file

@ -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 @@ int http_request(struct http_client_ctx *ctx,
}
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;