#16361: IGNORABLE_404 functionality should respect APPEND_SLASH setting
------------------------------------+-------------------------------
Reporter: Leo | Owner: nobody
Type: Bug | Status: new
Milestone: | Component: Documentation
Version: 1.3 | Severity: Normal
Resolution: | Keywords:
Triage Stage: Accepted | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
------------------------------------+-------------------------------
Description changed by aaugustin:
Old 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.
New description:
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.
--
--
Ticket URL: <https://code.djangoproject.com/ticket/16361#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
--
You received this message because you are subscribed to the Google Groups
"Django updates" group.
To post to this group, send email to django-updates@googlegroups.com.
To unsubscribe from this group, send email to
django-updates+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en.
