altered internal loops of how ``SessionRedirectMixin.resolve_redirect…

HISTORY.rst

@@ -12,6 +12,9 @@ Release History
   querying ``Response.is_redirect`` and ``Response.headers['location']``.
   Advanced users will be able to process malformed redirects more easily.
 
+- Altered how ``SessionRedirectMixin.resolve_redirects`` and ``Session.send``
+  process redirect history.
+
 2.13.0 (2017-01-24)
 +++++++++++++++++++
 

requests/sessions.py

@@ -97,17 +97,12 @@ def resolve_redirects(self, resp, req, stream=False, timeout=None,
                           verify=True, cert=None, proxies=None, **adapter_kwargs):
         """Receives a Response. Returns a generator of Responses."""
 
-        hist = [] # keep track of history
+        hist = [resp, ] # keep track of history; seed it with the original response
 
         url = self.get_redirect_target(resp)
         while url:
             prepared_request = req.copy()
 
-            # Update history and keep track of redirects.
-            # resp.history must ignore the original request in this loop
-            hist.append(resp)
-            resp.history = hist[1:]
-
             try:
                 resp.content  # Consume socket so it can be released
             except (ChunkedEncodingError, ContentDecodingError, RuntimeError):
@@ -193,6 +188,10 @@ def resolve_redirects(self, resp, req, stream=False, timeout=None,
                 allow_redirects=False,
                 **adapter_kwargs
             )
+            # copy our history tracker into the response
+            resp.history = hist[:]
+            # append the new response to the history tracker for the next iteration
+            hist.append(resp)
 
             extract_cookies_to_jar(self.cookies, prepared_request, resp.raw)
 
@@ -637,13 +636,9 @@ def send(self, request, **kwargs):
         # Resolve redirects if allowed.
         history = [resp for resp in gen] if allow_redirects else []
 
-        # Shuffle things around if there's history.
+        # if there is a history, replace ``r`` with the last response
         if history:
-            # Insert the first (original) request at the start
-            history.insert(0, r)
-            # Get the last request made
             r = history.pop()
-            r.history = history
 
         if not stream:
             r.content