Modify ↓
#4878 closed defect (fixed)
Weird URL handling of additional characters on the end
| Reported by: | Owned by: | Remy Blank | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.11.2 |
| Component: | timeline | Version: | |
| Severity: | trivial | Keywords: | |
| Cc: | Branch: | ||
| Release Notes: | |||
| API Changes: | |||
| Internal Changes: | |||
Description
The following URL should not work:
http://trac.edgewall.org/timeline/bogus
It should either return a 4040 status or a permanent redirect to /timeline
Attachments (1)
Change History (6)
comment:1 by , 18 years ago
| Component: | general → timeline |
|---|---|
| Milestone: | → 0.11 |
| Owner: | changed from to |
| Severity: | normal → trivial |
comment:2 by , 18 years ago
comment:3 by , 16 years ago
| Owner: | changed from to |
|---|
I'll try to implement the suggestions discussed in this thread, more specifically:
- Be more restrictive in URL matching.
- Redirect "{url}/" to "{url}" when no handler matches the former.
by , 16 years ago
| Attachment: | 4878-stricter-url-matching-r7510.patch added |
|---|
Patch against 0.11-stable adding stricter URL matching and redirects for URLs with trailing /
comment:4 by , 16 years ago
The patch above implements stricter URL matching, and redirects URLs with a trailing / for which no handler has been found. All unit tests pass.
Comments?
comment:5 by , 16 years ago
| Milestone: | 0.11.3 → 0.11.2 |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Patch applied in [7549].
Note:
See TracTickets
for help on using tickets.
Please note that this effects all areas of Trac - not just the timeline.