search

TrafeX / docker-php-nginx

Docker image with PHP-FPM 8.3 & Nginx 1.24 on Alpine Linux
https://hub.docker.com/r/trafex/php-nginx
MIT License
1.33k stars 721 forks source link

Update Dockerfile with Laravel PHP extensions needed #149

Closed insign closed 9 months ago

insign commented 9 months ago

Based on https://laravel.com/docs/10.x/deployment#server-requirements

I think it is relevant since is the most popular framework these days.

xmlwriter is needed for Laravel Sanctum.

insign commented 9 months ago

Related to #148

TrafeX commented 9 months ago

Hi @insign ,

Thank you for your contribution. Although I hold back on adding extensions to keep the base image as small as possible, these extensions are part of what's bundled with PHP (except for xmlwriter, but we also have xmlreader so it's not adding any overhead). So I'll be happy to merge this!

insign commented 9 months ago

I truly understand that you want a very simple and clean image but have composer looks like a must. I saw in the documentation how to add but keeps the sensation that is so needed because even small projects should rely on composer to keeps things tidy.

insign commented 9 months ago

And... like others images since the deploy is automated could have a image-dev with debug enable by default (even I not use it)