From: Daniel Stenberg Date: Thu, 3 Apr 2008 21:44:49 +0000 (+0000) Subject: - Setting CURLOPT_NOBODY to FALSE will now switch the HTTP request method to X-Git-Tag: curl-7_18_2~190 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=a9c1ca9fc5a80e858b0c85d8f8c838b4464526db;p=thirdparty%2Fcurl.git - Setting CURLOPT_NOBODY to FALSE will now switch the HTTP request method to GET simply because previously when you set CURLOPT_NOBODY to TRUE first and then FALSE you'd end up in a broken state where a HTTP request would do a HEAD by still act a lot like for a GET and hang waiting for the content etc. --- diff --git a/CHANGES b/CHANGES index e55bd540ad..e11ae621a7 100644 --- a/CHANGES +++ b/CHANGES @@ -7,6 +7,11 @@ Changelog Daniel Stenberg (3 Apr 2008) +- Setting CURLOPT_NOBODY to FALSE will now switch the HTTP request method to + GET simply because previously when you set CURLOPT_NOBODY to TRUE first and + then FALSE you'd end up in a broken state where a HTTP request would do a + HEAD by still act a lot like for a GET and hang waiting for the content etc. + - Scott Barrett added support for CURLOPT_NOBODY over SFTP Daniel Fandrich (3 Apr 2008) diff --git a/RELEASE-NOTES b/RELEASE-NOTES index 19a053ea1e..912983ad0a 100644 --- a/RELEASE-NOTES +++ b/RELEASE-NOTES @@ -15,7 +15,8 @@ This release includes the following changes: This release includes the following bugfixes: - o + o CURLOPT_NOBODY first set to TRUE and then FALSE for HTTP no longer causes + the confusion that could lead to a hung transfer This release includes the following known bugs: diff --git a/TODO-RELEASE b/TODO-RELEASE index 1a6e188460..650e8915dc 100644 --- a/TODO-RELEASE +++ b/TODO-RELEASE @@ -7,7 +7,4 @@ To be addressed before 7.18.2 (planned release: June 2008) 130 - Vincent Le Normand's SFTP patch for touch (lacking feedback) -133 - Setting CURLOPT_NOBODY to "false" causes cURL to wait for content if a - content-length header is read - 134 - diff --git a/lib/url.c b/lib/url.c index 9d782359b5..92f7156d33 100644 --- a/lib/url.c +++ b/lib/url.c @@ -858,9 +858,13 @@ CURLcode Curl_setopt(struct SessionHandle *data, CURLoption option, * Do not include the body part in the output data stream. */ data->set.opt_no_body = (bool)(0 != va_arg(param, long)); - if(data->set.opt_no_body) - /* in HTTP lingo, this means using the HEAD request */ - data->set.httpreq = HTTPREQ_HEAD; + + /* in HTTP lingo, no body means using the HEAD request and if unset there + really is no perfect method that is the "opposite" of HEAD but in + reality most people probably think GET then. The important thing is + that we can't let it remain HEAD if the opt_no_body is set FALSE since + then we'll behave wrong when getting HTTP. */ + data->set.httpreq = data->set.opt_no_body?HTTPREQ_HEAD:HTTPREQ_GET; break; case CURLOPT_FAILONERROR: /*