motioneye-project/motioneye

Example Config is IPv4-only, yet tornado supports IPv6

Offen

#779 geöffnet am 29.01.2018

 (25 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Python (636 Forks)batch import
help wantedquestion

Repository-Metriken

Stars
 (3.533 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 16T 14h) (33 gemergte PRs in 30 T)

Beschreibung

Given this day in age where IPv4 addresses are exhausted, and 20% of the traffic on the internet is IPv6, it would be great to see MotionEye support IPv6 as well.

Looking into the code a bit, it appears that a small change to the settings.py will enable MotionEye to support IPv6 (at least for the control webserver). The current settings.py has:

# the IP address to listen on
# (0.0.0.0 for all interfaces, 127.0.0.1 for localhost)
LISTEN = '0.0.0.0'

Yet, setting LISTEN to an empty string will enable the tornado webserver to listen on both IPv4 AND IPv6

# the IP address to listen on
# (0.0.0.0 or :: for all interfaces, 127.0.0.1 or ::1 for localhost, blank for both IPv4 & IPv6)
LISTEN = ''

At a minimum it would be nice if the comments reflected support for IPv6, and perhaps the default settings.py could be a blank string, enabling both IPv4 AND IPv6 support for your fine software.

Contributor Guide