Opened 13 years ago
Last modified 10 years ago
#16361 closed Bug
IGNORABLE_404 functionality should respect APPEND_SLASH setting — at Initial Version
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
I was originally going to file this against IGNORABLE_404_ENDS, but it looks like #16160 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.