Opened 14 years ago
Last modified 11 years ago
#16361 closed Bug
IGNORABLE_404 functionality should respect APPEND_SLASH setting — at Version 3
| Reported by: | Leo Shklovskii | Owned by: | nobody |
|---|---|---|---|
| Component: | Documentation | Version: | 1.3 |
| Severity: | Normal | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description (last modified by )
I was originally going to file this against IGNORABLE_404_ENDS, but it looks like r16160 rewrote that setting and the way these are configured, however, the bug still remains.
When APPEND_SLASH is enabled, CommonMiddleware issues the redirect on /favicon.ico and a 404 for /favicon.ico/ ends up showing up in the logs.
If it's a pain to fix this issue given the new implementation, one alternative is to just add a note to the docs telling people to add a //? to the end of their regex.
Change History (3)
comment:2 by , 14 years ago
| Component: | Core (Other) → Documentation |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
Accepted as a documentation fix.
comment:3 by , 14 years ago
| Description: | modified (diff) |
|---|
Oops, sorry, that should have been r16160 above.
edit: just fixed in the summary.