IGNORABLE_404 functionality should respect APPEND_SLASH setting
|Reported by:||Leo Shklovskii||Owned by:||nobody|
|Has patch:||no||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||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 (8)
comment:1 Changed 5 years ago by
|Patch needs improvement:||unset|
comment:2 Changed 5 years ago by
|Component:||Core (Other) → Documentation|
|Triage Stage:||Unreviewed → Accepted|