Merge pull request #2244 from sigmavirus24/bug/2231

Avoid getting stuck in a loop
2026-06-05 22:50:18 +00:00 · 2014-09-24 22:57:38 -04:00
parent e0d88c2814 ac44b48297
commit de76fbeefb
1 changed files with 6 additions and 1 deletions
@@ -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)