[Django] #31319: urldispatcher does not handle multishlashes

9 views
Skip to first unread message

Django

unread,
Feb 28, 2020, 10:11:25 AM2/28/20
to django-...@googlegroups.com
#31319: urldispatcher does not handle multishlashes
-----------------------------------------+------------------------
Reporter: Bjoern | Owner: nobody
Type: Uncategorized | Status: new
Component: Uncategorized | Version: 3.0
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+------------------------
while removing nginx in my setup I discovered that the url dispatcher does
not seem to handle path names correctly

e.g.
{{{
/some/fancy/endpoint
vs
/some///fancy/endpoint
}}}

nginx did "normalize" that url like a unix system would do
`cd /usr////bin` still brings you to /usr/bin

not sure if it is intensionally, but I'd assume django would/should handle
it the same

--
Ticket URL: <https://code.djangoproject.com/ticket/31319>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Mar 2, 2020, 2:55:53 AM3/2/20
to django-...@googlegroups.com
#31319: URL dispatcher does not handle multislashes.
-----------------------------+--------------------------------------
Reporter: Bjoern | Owner: nobody
Type: New feature | Status: closed
Component: Core (URLs) | Version: 3.0
Severity: Normal | Resolution: wontfix

Keywords: | Triage Stage: Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-----------------------------+--------------------------------------
Changes (by felixxm):

* status: new => closed
* resolution: => wontfix
* type: Uncategorized => New feature
* component: Uncategorized => Core (URLs)


Comment:

Thanks for this ticket, however I don't agree that URL dispatcher should
merge slashes. This change would be backward incompatible and IMO it's
something that lies within responsibility of a web server (like nginx).
You can start a discussion on DevelopersMailingList if you don't agree.

--
Ticket URL: <https://code.djangoproject.com/ticket/31319#comment:1>

Reply all
Reply to author
Forward
0 new messages