[Python-checkins] r80584 - in python/branches/release26-maint: Doc/library/httplib.rst Lib/httplib.py Lib/test/test_httplib.py Misc/NEWS

[senthil.kumaran]

Author: senthil.kumaran
Date: Wed Apr 28 19:25:58 2010
New Revision: 80584

Log:
Merged revisions 80583 via svnmerge from 
svn+ssh://pythondev@svn.python.org/python/trunk

........
  r80583 | senthil.kumaran | 2010-04-28 22:50:43 +0530 (Wed, 28 Apr 2010) | 3 lines
  
  Fixed Issue6312 - httplib fails with HEAD requests to pages with "transfer-encoding: chunked"
........


Modified:
   python/branches/release26-maint/   (props changed)
   python/branches/release26-maint/Doc/library/httplib.rst
   python/branches/release26-maint/Lib/httplib.py
   python/branches/release26-maint/Lib/test/test_httplib.py
   python/branches/release26-maint/Misc/NEWS

Modified: python/branches/release26-maint/Doc/library/httplib.rst
==============================================================================
--- python/branches/release26-maint/Doc/library/httplib.rst	(original)
+++ python/branches/release26-maint/Doc/library/httplib.rst	Wed Apr 28 19:25:58 2010
@@ -541,6 +541,22 @@
    >>> data2 = r2.read()
    >>> conn.close()
 
+Here is an example session that uses ``HEAD`` method. Note that ``HEAD`` method
+never returns any data. ::
+
+
+   >>> import httplib
+   >>> conn = httplib.HTTPConnection("www.python.org")
+   >>> conn.request("HEAD","/index.html")
+   >>> res = conn.getresponse()
+   >>> print res.status, res.reason
+   200 OK
+   >>> data = res.read()
+   >>> print len(data)
+   0
+   >>> data == ''
+   True
+
 Here is an example session that shows how to ``POST`` requests::
 
    >>> import httplib, urllib

Modified: python/branches/release26-maint/Lib/httplib.py
==============================================================================
--- python/branches/release26-maint/Lib/httplib.py	(original)
+++ python/branches/release26-maint/Lib/httplib.py	Wed Apr 28 19:25:58 2010
@@ -514,6 +514,9 @@
         if self.fp is None:
             return ''
 
+        if self._method == 'HEAD':
+            return ''
+
         if self.chunked:
             return self._read_chunked(amt)
 

Modified: python/branches/release26-maint/Lib/test/test_httplib.py
==============================================================================
--- python/branches/release26-maint/Lib/test/test_httplib.py	(original)
+++ python/branches/release26-maint/Lib/test/test_httplib.py	Wed Apr 28 19:25:58 2010
@@ -207,6 +207,23 @@
             finally:
                 resp.close()
 
+    def test_chunked_head(self):
+        chunked_start = (
+            'HTTP/1.1 200 OK\r\n'
+            'Transfer-Encoding: chunked\r\n\r\n'
+            'a\r\n'
+            'hello world\r\n'
+            '1\r\n'
+            'd\r\n'
+        )
+        sock = FakeSocket(chunked_start + '0\r\n')
+        resp = httplib.HTTPResponse(sock, method="HEAD")
+        resp.begin()
+        self.assertEquals(resp.read(), '')
+        self.assertEquals(resp.status, 200)
+        self.assertEquals(resp.reason, 'OK')
+        resp.close()
+
     def test_negative_content_length(self):
         sock = FakeSocket('HTTP/1.1 200 OK\r\n'
                           'Content-Length: -1\r\n\r\nHello\r\n')

Modified: python/branches/release26-maint/Misc/NEWS
==============================================================================
--- python/branches/release26-maint/Misc/NEWS	(original)
+++ python/branches/release26-maint/Misc/NEWS	Wed Apr 28 19:25:58 2010
@@ -33,6 +33,9 @@
 Library
 -------
 
+- Issue #6312: Fixed http HEAD request when the transfer encoding is chunked.
+  It should correctly return an empty response now.
+
 - Issue #8086: In :func:`ssl.DER_cert_to_PEM_cert()`, fix missing newline
   before the certificate footer.  Patch by Kyle VanderBeek.