Opened 5 years ago

Closed 5 years ago

#31319 closed New feature (wontfix)

URL dispatcher does not handle multislashes.

Reported by: Bjoern Owned by: nobody
Component: Core (URLs) Version: 3.0
Severity: Normal Keywords:
Cc: Triage Stage: Unreviewed
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

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

Change History (1)

comment:1 by Mariusz Felisiak, 5 years ago

Component: UncategorizedCore (URLs)
Resolution: wontfix
Status: newclosed
Summary: urldispatcher does not handle multishlashesURL dispatcher does not handle multislashes.
Type: UncategorizedNew feature

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.

Note: See TracTickets for help on using tickets.
Back to Top