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.