search

serversideup / docker-php

🐳 Production-ready Docker images for PHP. Optimized for Laravel, WordPress, and more!
https://serversideup.net/open-source/docker-php/
GNU General Public License v3.0
1.65k stars 108 forks source link

Nginx Unit Docker Image Exposes Ports 80 and 443 alongside 8080 and 8443? #355

Closed jaydrogers closed 3 months ago

jaydrogers commented 3 months ago

I'm testing the serversideup/php:8.3-unit image on a server behind Traefik and am having trouble getting everything to work. The Ngnix Unit image is building successfully and responding on port 8080 for Laravel 11 health check. But Traefik seems to be discovering port 80 instead of 8080 for routing traffic to the service.

In looking at the Dockerfile for the Nginx Unit variation (src/variations/unit/Dockerfile), I see that it exposes 80 and 443 on line 151 and exposes 8080 and 8443 on line 163. Is this correct? It's different from the Nginx FPM Dockerfile (src/variations/fpm-nginx/Dockerfile), which only exposes 8080 and 8443 (line 173). It also doesn't seem to match the documentation listed on https://serversideup.net/open-source/docker-php/docs/getting-started/default-configurations.

Originally posted by @jpangborn in https://github.com/serversideup/docker-php/discussions/349

jaydrogers commented 3 months ago

Thanks for reporting this @jpangborn!

I just merged this into our development images.

Watch the releases for once this is available 🎉