#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 , 10 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 , 10 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.debugto add SQL queries to context. Its docs mention the setting. Is it enough?django.contrib.admindocs.middleware.XViewMiddlewareto 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.AdminEmailHandlerto 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 , 10 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
comment:4 by , 10 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 , 10 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 , 10 years ago
| Has patch: | set |
|---|
There's also a usage in
AdminEmailHandlerthat could be mentioned.