Skip to content

Conversation

@yuuzukatsu
Copy link
Contributor

Hello,

I've been running into some problem with the default configuration listening to IPv6. I was hoping with this env, it enables user to toggle whether they want to listen on IPv6 address or not

image

adding NGINX_IPV6=false environment will delete line containing listen [::]: on both /etc/nginx/site-opts.d/http.conf and /etc/nginx/site-opts.d/https.conf

I've tested this entrypoint script changes specifically in 8.2-fpm-nginx but adding few lines like this should work in most nginx variation

Thank you

@tomschlick
Copy link
Contributor

It might be a good idea to add NGINX_IPV4 toggle to this as well. Many K8s environments are IPv6 only (LB / NAT GW handling IPv4 ingress/egress) and could benefit from turning it off.

@jaydrogers jaydrogers moved this to 🤔 Evaluating in serversideup/php v4.0 Sep 23, 2025
@jaydrogers jaydrogers changed the base branch from main to 280-create-a-frankenphp-variation September 23, 2025 15:35
jaydrogers and others added 2 commits October 1, 2025 13:00
…rotocol for NGINX. Update entrypoint script to handle IPv4 and IPv6 settings based on this variable. Enhance documentation to reflect the new configuration option.
@jaydrogers jaydrogers changed the title FEAT: add NGINX_IPV6 env to toggle nginx IPv6 listening FEAT: add NGINX_IP_LISTENING_PROTOCOL Oct 1, 2025
@jaydrogers
Copy link
Member

Thank you @yuuzukatsu for the idea on this, and thanks to @tomschlick for chiming in with more use cases.

Instead of NGINX_IPV6 being a boolean, I created NGINX_LISTEN_IP_PROTOCOL (default all). You can switch it between all, ipv4, and ipv6 to get your desired results.

This will be made available in V4.0:

@jaydrogers jaydrogers merged commit f4d237f into serversideup:280-create-a-frankenphp-variation Oct 1, 2025
@github-project-automation github-project-automation bot moved this from 🤔 Evaluating to Done in serversideup/php v4.0 Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants