Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #1719: resolving hostnames for bind #1723

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Issue #1719: resolving hostnames for bind #1723

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
f3ceae2
Using getaddrinfo to resolve hostnames properly
wanneut Mar 12, 2018
b1779b5
better style
wanneut Mar 13, 2018
61ea13e
fixed socket.close
wanneut Mar 14, 2018
f0f2928
spcaes
wanneut Mar 14, 2018
06bd5d4
adopted some tests
wanneut Mar 14, 2018
3f633ec
reseted requirements
wanneut Mar 14, 2018
0c45c4a
more reasonable defaults: [::] and localhost will lsiten on IPv6+IPv4
wanneut Mar 14, 2018
25935b7
127.0.0.1 -> localhost
wanneut Mar 14, 2018
4874c1b
_sock_type gets only family
wanneut Mar 15, 2018
2bdd721
adopted last test
wanneut Mar 15, 2018
c94048e
IPv6 schme handling, test coverage
Mar 15, 2018
d3568c8
test for tcp://
wanneut Mar 15, 2018
7904f3f
Removed test depending on modern /etc/hosts
wanneut Apr 18, 2018
0ebf15c
accept IPv4 only environment
wanneut Apr 18, 2018
6f91687
rerun checks
wanneut Apr 25, 2018
51466dc
making code unreadable
wanneut Apr 25, 2018
c597560
removed trailing whitespace
wanneut Apr 25, 2018
4c20ec1
-
wanneut Apr 25, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions gunicorn/config.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -538,9 +538,9 @@ class Bind(Setting):
validator = validate_list_string

if 'PORT' in os.environ:
default = ['0.0.0.0:{0}'.format(os.environ.get('PORT'))]
default = ['[::]:{0}'.format(os.environ.get('PORT'))]
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At least on my machine, this returns only IPv6 addresses from socket.getaddrinfo(). Since this is a list, maybe we want to give it two entries so Gunicorn binds to IPv4 and IPv6 by default?

Copy link
Author

@wanneut wanneut Mar 15, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you bind to :: you will listen to any address (IPv4 and IPv6) as long the IPV6_V6ONLY flag is not set.
In fact :: could be written as ::0.0.0.0 which is already the IPv4-compatible address for 0.0.0.0. (Older version of ::FFFF:0.0.0.0)

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about the machine that doesn't support or has not been installed with ipv6 support?

Copy link
Author

@wanneut wanneut Apr 3, 2018 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The older version dosn't work on IPv6 installations at all.
This version, works everywhere (IPv4 like IPv6) exempt of some obscure Linux-configurations (I think there are not many other OSes where something can be done.), where someone explicitly removed IPv6 support. I think such users should know how to specify another bind address.
If someone shoots himself into his foot he should not wonder if it hurts.
Gunicorn requires Python 2 (which is newer than IPv6.) multithreading and several other things which are much more uncommon than IPv6.
At the end you have to decide what your defaults are.
I mean what about installations with disabled IPv4? It is deprecated for 20 years now. Time to turn it off? IPv6 is compatible to it. So it shouldn't be needed any more. The new code could handle that.

else:
default = ['127.0.0.1:8000']
default = ['localhost:8000']

desc = """\
The socket to bind.
Expand Down