[Django] #24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error

18 views
Skip to first unread message

Django

unread,
Jun 10, 2015, 4:51:56 PM6/10/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
--------------------------------------+---------------------
Reporter: raiderrobert | Owner:
Type: Cleanup/optimization | Status: new
Component: Uncategorized | Version: 1.8
Severity: Normal | Keywords: setting
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
--------------------------------------+---------------------
Allowed_Hosts is a required setting. When debug mode is off, and
Allowed_Hosts is not implemented, this should raise an
ImproperlyConfigured Error.

See conversation here: https://groups.google.com/forum/#!topic/django-
developers/nIrbGjyOsNY

--
Ticket URL: <https://code.djangoproject.com/ticket/24966>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Jun 10, 2015, 4:52:27 PM6/10/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Uncategorized | Version: 1.8
Severity: Normal | Resolution:

Keywords: setting | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by raiderrobert):

* owner: => raiderrobert
* needs_better_patch: => 0
* status: new => assigned
* needs_tests: => 0
* needs_docs: => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:1>

Django

unread,
Jun 10, 2015, 4:56:12 PM6/10/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: setting | Triage Stage: Accepted

Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by carljm):

* component: Uncategorized => Core (Other)
* stage: Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:2>

Django

unread,
Jun 16, 2015, 4:07:10 PM6/16/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: setting | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by raiderrobert):

Going to make it a warning instead of a hard fail. It's going in
`core/checks/security/base.py`.

--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:3>

Django

unread,
Jun 19, 2015, 9:18:33 AM6/19/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: setting | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* needs_docs: 0 => 1
* has_patch: 0 => 1
* needs_tests: 0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:4>

Django

unread,
Jul 12, 2015, 3:40:57 PM7/12/15
to django-...@googlegroups.com
#24966: Unconfigured Allowed_Hosts Setting Should Raise ImproperlyConfigured Error
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: setting | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1
Needs tests: 1 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by sggottlieb):

Submitted a pull request to raiderrobert that adds documentation for his
pull request.

--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:5>

Django

unread,
Jul 14, 2015, 10:35:39 PM7/14/15
to django-...@googlegroups.com
#24966: Add deployment system check for empty ALLOWED_HOSTS setting

-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: assigned
Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution:
Keywords: setting | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by timgraham):

* needs_docs: 1 => 0
* needs_tests: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:6>

Django

unread,
Jul 15, 2015, 9:20:26 AM7/15/15
to django-...@googlegroups.com
#24966: Add deployment system check for empty ALLOWED_HOSTS setting
-------------------------------------+-------------------------------------
Reporter: raiderrobert | Owner:
Type: | raiderrobert
Cleanup/optimization | Status: closed

Component: Core (Other) | Version: 1.8
Severity: Normal | Resolution: fixed

Keywords: setting | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham <timograham@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"ed514caed20995814712fd6c9e9c4120b4ac64be" ed514cae]:
{{{
#!CommitTicketReference repository=""
revision="ed514caed20995814712fd6c9e9c4120b4ac64be"
Fixed #24966 -- Added deployment system check for empty ALLOWED_HOSTS.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/24966#comment:7>

Reply all
Reply to author
Forward
0 new messages