-
-
Notifications
You must be signed in to change notification settings - Fork 38
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.
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
Enable IPv6 support on uwsgi #123
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for opening this pull request! Be sure to follow the pull request template!
How does uwsgi handle this on hosts with ipv6 disabled in the kernel? |
It breaks, as with anything using dual IPv4/IPv6 sockets. But in a choice between supporting IPv6 and supporting setups with IPv6 intentionally disabled, we should probably favour the former. Also it doesn't appear possible to bind the smtpd service to IPv6 in either case. This needs adding to the readme changelog; it's a significant change. |
I am a bot, here are the test results for this PR:
|
On a non-ipv6 network, everything works as it was: Create ipv4 only test network
Executing the homebuild container
Inspecting the test network
As you can see, only ipv4. Checking inside the container
Starting a test nginx container on the same network
Connecting over the bridge network to the other container on ipv4
|
|
People still do that? I'll see to spin up a machine with ipv6 disabled but that'll take a bit. |
With alarming regularity |
Tested on a host without ipv6 enabled - it breaks spectacularly as stated. |
I am a bot, here are the test results for this PR:
|
No, don't explicitly mention it breaking in the changelog; we'll see if anyone is actually affected and if there are sufficient numbers we may need to add a conditional fudge for them, but we try and avoid it when possible. |
I am a bot, here are the test results for this PR:
|
Hello edit: I tried to pass env var
|
For those who would face the same issue and since nobody answered here, the only solution until now is to manually override |
Closes #122
Description:
Enable IPV6 on uwsgi
Benefits of this PR and context:
Allows the container to be accessed over ipv6 inside the docker network.
How Has This Been Tested?
Built the container, tested it on my local installation.
Source / References: