#25710 closed Cleanup/optimization (fixed)
Clarify the docs about what INTERNAL_IPS does
Reported by: | Marten Kenbeek | Owned by: | Alex Morozov |
---|---|---|---|
Component: | Documentation | Version: | 1.8 |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Accepted | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | yes | UI/UX: | no |
Description
The current documentation says:
A tuple of IP addresses, as strings, that:
- See debug comments, when DEBUG is True
- Receive X headers in admindocs if the XViewMiddleware is installed (see The Django admin documentation generator)
The documentation of DEBUG
doesn't mention INTERNAL_IPS
at all, and the documentation of the admindocs only briefly mentions it in the last line.
Change History (11)
comment:1 by , 9 years ago
Easy pickings: | set |
---|---|
Summary: | Document what INTERNAL_IPS does → Clarify the docs about what INTERNAL_IPS does |
Triage Stage: | Unreviewed → Accepted |
comment:2 by , 9 years ago
Guys, could you please elaborate on what specifically must be documented?
INTERNAL_IPS
is now used at three places (except tests):
django.template.context_processors.debug
to add SQL queries to context. Its docs mention the setting. Is it enough?django.contrib.admindocs.middleware.XViewMiddleware
to add an X-View header. Docs mention the setting. Should we refactor documentation? I hardly can imagine what to add about the middleware itself.django.utils.log.AdminEmailHandler
to prefix an email subject with an 'internal'/'EXTERNAL' marks. Should we mention the setting here?
As for the DEBUG
documentation, it seems pretty complete to me, since this setting has no direct relation to the INTERNAL_IPS
. Am I wrong?
comment:3 by , 9 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:4 by , 9 years ago
Linking from the INTERNAL_IPS setting to where each of those features are discussed in more detail would be helpful.
comment:5 by , 9 years ago
@timgraham, thank you. Do I understand correctly that the existing phrase "See debug comments, when DEBUG
is True
" is about the debug
context processor?
comment:8 by , 9 years ago
Has patch: | set |
---|
There's also a usage in
AdminEmailHandler
that could be mentioned.