Implement history for ClientResponseError.

CHANGES.rst

@@ -10,6 +10,8 @@ Changes
 
 - Dropped "%O" in access logger #1673
 
+- Added `history` to `ClientResponseError`. #1741
+
 
 2.0.2 (2017-03-21)
 ------------------

aiohttp/client_exceptions.py

@@ -25,11 +25,13 @@ class ClientResponseError(ClientError):
     :param request_info: instance of RequestInfo
     """
 
-    def __init__(self, request_info, *, code=0, message='', headers=None):
+    def __init__(self, request_info, *, code=0, message='', headers=None,
+                 history=None):
         self.request_info = request_info
         self.code = code
         self.message = message
         self.headers = headers
+        self.history = history if history is not None else ()
 
         super().__init__("%s, message='%s'" % (code, message))
 

aiohttp/client_reqrep.py

@@ -548,7 +548,8 @@ def start(self, connection, read_until_eof=False):
                     raise ClientResponseError(
                         self.request_info,
                         code=exc.code,
-                        message=exc.message, headers=exc.headers) from exc
+                        message=exc.message, headers=exc.headers,
+                        history=self.history) from exc
 
                 if (message.code < 100 or
                         message.code > 199 or message.code == 101):
@@ -634,7 +635,8 @@ def raise_for_status(self):
                 self.request_info,
                 code=self.status,
                 message=self.reason,
-                headers=self.headers)
+                headers=self.headers,
+                history=self.history)
 
     def _cleanup_writer(self):
         if self._writer is not None and not self._writer.done():
@@ -709,7 +711,8 @@ def json(self, *, encoding=None, loads=json.loads,
                     self.request_info,
                     message=('Attempt to decode JSON with '
                              'unexpected mimetype: %s' % ctype),
-                    headers=self.headers)
+                    headers=self.headers,
+                    history=self.history)
 
         stripped = self._content.strip()
         if not stripped:

docs/client_reference.rst

@@ -1335,6 +1335,9 @@ Hierarchy of exceptions:
    .. attribute:: request_info
                   Instance of `RequestInfo` object, contains information about request.
 
+   .. attribute:: history
+                  History from `ClientResponse` object, if available, else empty tuple.
+
   * `WSServerHandshakeError` - web socket server response error
 
     - `ClientHttpProxyError` - proxy response

tests/test_client_response.py

@@ -491,3 +491,61 @@ def test_request_info_in_exception():
     with pytest.raises(aiohttp.ClientResponseError) as cm:
         response.raise_for_status()
     assert cm.value.request_info == response.request_info
+
+
+def test_no_redirect_history_in_exception():
+    url = 'http://def-cl-resp.org'
+    headers = {'Content-Type': 'application/json;charset=cp1251'}
+    response = ClientResponse(
+        'get',
+        URL(url),
+        request_info=RequestInfo(
+            url,
+            'get',
+            headers
+        )
+    )
+    response.status = 409
+    response.reason = 'CONFLICT'
+    with pytest.raises(aiohttp.ClientResponseError) as cm:
+        response.raise_for_status()
+    assert () == cm.value.history
+
+
+def test_redirect_history_in_exception():
+    hist_url = 'http://def-cl-resp.org'
+    url = 'http://def-cl-resp.org/index.htm'
+    hist_headers = {'Content-Type': 'application/json;charset=cp1251',
+                    'Location': url
+                    }
+    headers = {'Content-Type': 'application/json;charset=cp1251'}
+    response = ClientResponse(
+        'get',
+        URL(url),
+        request_info=RequestInfo(
+            url,
+            'get',
+            headers
+        )
+    )
+    response.status = 409
+    response.reason = 'CONFLICT'
+
+    hist_response = ClientResponse(
+        'get',
+        URL(hist_url),
+        request_info=RequestInfo(
+            url,
+            'get',
+            headers
+        )
+    )
+
+    hist_response.headers = hist_headers
+    hist_response.status = 301
+    hist_response.reason = 'REDIRECT'
+
+    response._history = [hist_response]
+    with pytest.raises(aiohttp.ClientResponseError) as cm:
+        response.raise_for_status()
+    assert [hist_response] == cm.value.history