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 , 5 years ago
Component: | Uncategorized → Core (URLs) |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
Summary: | urldispatcher does not handle multishlashes → URL dispatcher does not handle multislashes. |
Type: | Uncategorized → New feature |
Note:
See TracTickets
for help on using tickets.
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.