#26428: Add support for relative path redirects to the test Client
-----------------------------------+------------------------------------
Reporter: master | Owner: nobody
Type: New feature | Status: closed
Component: Testing framework | Version: 1.9
Severity: Release blocker | Resolution: fixed
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-----------------------------------+------------------------------------
Comment (by Jacob):
This issue was not fully addressed. There are still a couple problems:
1) It assumes that a relative redirect does not start with '/'. But what
if you redirect relative to host with a '/', as in redirecting to
'/subpath' instead of 'https://example.com/subpath'?
2) It does not set the secure kwarg on the client.get call based on the
original host scheme, because urlsplit is called only on the relative
path. It seems that urlsplit should be called after building an absolute
url.
Should this ticket be re-opened, or a separate ticket created?
--
Ticket URL: <https://code.djangoproject.com/ticket/26428#comment:12>
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/064.1670bb61a150b08af8ce69eecaf14aa1%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
