From 207cf15032a7e8eab71b2f4655fc5c21e5d3623d Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sat, 31 Dec 2011 10:39:54 +0100 Subject: changed case: use new host name for subsequent HTTP requests When a HTTP connection is re-used for a subsequent request without proxy, it would always re-use the Host: header of the first request. As host names are case insensitive it would make curl send another host name case that what the particular request used. Now it will instead always use the most recent host name to always use the desired casing. Added test case 1318 to verify. Bug: http://curl.haxx.se/mail/lib-2011-12/0314.html Reported by: Alex Vinnik --- lib/url.c | 12 ++++------ tests/data/Makefile.am | 2 +- tests/data/test1318 | 59 ++++++++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 64 insertions(+), 9 deletions(-) create mode 100644 tests/data/test1318 diff --git a/lib/url.c b/lib/url.c index a95426fe6..86a5576c8 100644 --- a/lib/url.c +++ b/lib/url.c @@ -4666,14 +4666,10 @@ static void reuse_conn(struct connectdata *old_conn, old_conn->proxypasswd = NULL; } - /* host can change, when doing keepalive with a proxy ! */ - if(conn->bits.proxy) { - Curl_safefree(conn->host.rawalloc); - conn->host=old_conn->host; - } - else - /* free the newly allocated name buffer */ - Curl_safefree(old_conn->host.rawalloc); + /* host can change, when doing keepalive with a proxy or if the case is + different this time etc */ + Curl_safefree(conn->host.rawalloc); + conn->host=old_conn->host; /* persist connection info in session handle */ Curl_persistconninfo(conn); diff --git a/tests/data/Makefile.am b/tests/data/Makefile.am index 1de2479df..53d94841f 100644 --- a/tests/data/Makefile.am +++ b/tests/data/Makefile.am @@ -81,7 +81,7 @@ test1208 test1209 test1210 test1211 \ test1220 \ test1300 test1301 test1302 test1303 test1304 test1305 \ test1306 test1307 test1308 test1309 test1310 test1311 test1312 test1313 \ -test1314 test1315 test1317 \ +test1314 test1315 test1317 test1318 \ test2000 test2001 test2002 test2003 test2004 EXTRA_DIST = $(TESTCASES) DISABLED diff --git a/tests/data/test1318 b/tests/data/test1318 new file mode 100644 index 000000000..709e08dc3 --- /dev/null +++ b/tests/data/test1318 @@ -0,0 +1,59 @@ + + + +HTTP +HTTP GET + + + +# +# Server-side + + +HTTP/1.1 200 OK +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake +Content-Length: 0 + + + +HTTP/1.1 200 second version +Date: Thu, 09 Nov 2010 14:49:00 GMT +Server: test-server/fake +Content-Length: 0 + + + + +# +# Client-side + + +http + + +HTTP with same host name using different cases + + +--resolve MiXeDcAsE.cOm:%HTTPPORT:%HOSTIP http://MiXeDcAsE.cOm:%HTTPPORT/1318 http://mixedcase.com:%HTTPPORT/13180001 + + + +# +# Verify data after the test has been "shot" + + +^User-Agent:.* + + +GET /1318 HTTP/1.1 +Host: MiXeDcAsE.cOm:%HTTPPORT +Accept: */* + +GET /13180001 HTTP/1.1 +Host: mixedcase.com:%HTTPPORT +Accept: */* + + + + -- cgit v1.2.3