#23682: Stronger redirect loop detection in the test client
-----------------------------------+--------------------
Reporter: wrwrwr | Owner: nobody
Type: Uncategorized | Status: new
Component: Testing framework | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
-----------------------------------+--------------------
The test client has a mechanism to detect circular redirects. I'd like to
propose two changes to it;
* when a loop is detected, throw an exception rather than break silently;
* detect and complain about very (possibly infinitely) long redirect
chains with differing URLs (at least when they only differ by a view
argument value or query string).
[https://github.com/wrwrwr/django/compare/dropped/redirect-with-view-name-
loops A few tests] that are not part of the proposed changes, but
demonstrate how one could try to test for loops (currently 3 of these just
past, while a browser would run into a loop, and 2 actually loop
indefinitely).
--
Ticket URL: <https://code.djangoproject.com/ticket/23682>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit
https://groups.google.com/d/msgid/django-updates/049.9ed867716e608272e023d2db1f672390%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
