#2461 closed enhancement (wontfix)
[patch] manage.py: Add support for IP, IP:PORT, or PORT command-line arguments
Reported by: | jvargas -a-t- acm -*- o r g | Owned by: | Adrian Holovaty |
---|---|---|---|
Component: | Core (Management commands) | Version: | dev |
Severity: | minor | Keywords: | ip address utils |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
These two files (iputils.py and iputils.diff) add support for IP, IP:PORT, or PORT command-line arguments to manage.py, so that
manage.py runserver [optional port number, or ipaddr:port]
becomes
manage.py runserver [optional ip address, optional port number, or ipaddr:port]
Attachments (2)
Change History (10)
by , 18 years ago
Attachment: | iputils.diff added |
---|
comment:2 by , 18 years ago
Summary: | manage.py: Add support for IP, IP:PORT, or PORT command-line arguments → [patch] manage.py: Add support for IP, IP:PORT, or PORT command-line arguments |
---|
Added [patch] prefix to ticket summary.
comment:3 by , 18 years ago
How is this different from what runserver
already does wrt to specifying port or IP/port? Have I missed something here?
comment:4 by , 18 years ago
Currently (SVN), you cannot do
$ python manage.py runserver 192.168.1.2
Error: '192.168.1.2' is not a valid port number.
comment:5 by , 18 years ago
Hm. So, really what this should be is just adding support for specifying only an IP and having it bind to the default port on that IP, so that runserver 192.168.1.2
would become functionally equivalent to runserver 192.168.1.2:8000
?
comment:6 by , 18 years ago
Indeed, but it also adds some checks for validity (see iputils.py file) and unit tests.
comment:7 by , 18 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
This doesn't introduce any new functionality, so I'm marking it as a wontfix. It's really not a big deal to have to specify the port if you want to use a custom IP address.
As an aside, I'm always hesitant to add more stuff to django.utils, so that fact played into this decision.
patch for management.py