Merge pull request #2244 from sigmavirus24/bug/2231

Avoid getting stuck in a loop

1 files changed, 6 insertions, 1 deletions

diff --git a/requests/sessions.py b/requests/sessions.py
index 023edfbd..f9873064 100644
--- a/requests/sessions.py
+++ b/requests/sessions.py
@@ -535,8 +535,13 @@ class Session(SessionRedirectMixin):
         if not isinstance(request, PreparedRequest):
             raise ValueError('You can only send PreparedRequests.')
 
+        checked_urls = set()
         while request.url in self.redirect_cache:
-            request.url = self.redirect_cache.get(request.url)
+            checked_urls.add(request.url)
+            new_url = self.redirect_cache.get(request.url)
+            if new_url in checked_urls:
+                break
+            request.url = new_url
 
         # Set up variables needed for resolve_redirects and dispatching of hooks
         allow_redirects = kwargs.pop('allow_redirects', True)