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.

Change History (0)

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