Opened 11 years ago

Closed 11 years ago

#3942 closed (fixed)

Trim leading slash from request.path on debug 404 page

Reported by: Collin Grady <cgrady@…> Owned by: Jacob
Component: Uncategorized Version: master
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:


At present, the debug 404 page lists the matches, which will not have leading slashes "foo/" for example, but it shows request.path with the leading slash - this can cause confusion to new users, as they might have "/foo/" and see it saying /foo/ didn't match.

With this patch, error message will list only the remainder, minus the leading slash.

Attachments (1)

debug.patch (843 bytes) - added by Collin Grady <cgrady@…> 11 years ago.

Download all attachments as: .zip

Change History (2)

Changed 11 years ago by Collin Grady <cgrady@…>

Attachment: debug.patch added

comment:1 Changed 11 years ago by Adrian Holovaty

Resolution: fixed
Status: newclosed

(In [4959]) Fixed #3942 -- Fixed potential confusion on debug 404 page by stripping initial slash from the display of the current URL. Thanks, Collin Grady

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