diff options
author | Daniel Stenberg <daniel@haxx.se> | 2006-11-25 13:32:04 +0000 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2006-11-25 13:32:04 +0000 |
commit | da58d03ff7be9fc5e0219d95ce2f9932e2272473 (patch) | |
tree | 8d2c198822b1642ac20c3cc7b387cd24e1bb4970 /lib | |
parent | 9ea3831c08fab5b320e381b1552964ff789316c7 (diff) |
Venkat Akella found out that libcurl did not like HTTP responses that simply
responded with a single status line and no headers nor body. Starting now, a
HTTP response on a persistent connection (i.e not set to be closed after the
response has been taken care of) must have Content-Length or chunked
encoding set, or libcurl will simply assume that there is no body.
To my horror I learned that we had no less than 57(!) test cases that did bad
HTTP responses like this, and even the test http server (sws) responded badly
when queried by the test system if it is the test system. So although the
actual fix for the problem was tiny, going through all the newly failing test
cases got really painful and boring.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/transfer.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/lib/transfer.c b/lib/transfer.c index 2d5d0dfe8..6b7b81978 100644 --- a/lib/transfer.c +++ b/lib/transfer.c @@ -501,9 +501,19 @@ CURLcode Curl_readwrite(struct connectdata *conn, k->keepon |= KEEP_WRITE; } } - else + else { k->header = FALSE; /* no more header to parse! */ + if((k->size == -1) && !conn->bits.chunk && !conn->bits.close) + /* When connection is not to get closed, but no + Content-Length nor Content-Encoding chunked have been + received, there is no body in this response. We don't set + stop_reading TRUE since that would also prevent necessary + authentication actions to take place. */ + conn->bits.no_body = TRUE; + + } + if (417 == k->httpcode) { /* * we got: "417 Expectation Failed" this means: |