search

webdevops / Dockerfile

:package: Dockerfiles from WebDevOps for PHP, Apache and Nginx
https://webdevops.io/projects/dockerfiles/
MIT License
1.67k stars 493 forks source link

update php version on webdevops/php:8.2 image #507

Open op-euga opened 9 months ago

op-euga commented 9 months ago

please update the php version. currently is uses 8.2.7 while webdevops/php:8.2-alpine already uses 8.2.14. It seem that the php version 8.2.7 has segmentation fault problems.

htuscher commented 9 months ago

The image for 8.2 is based on php:8.2-fpm-buster and this version contains only 8.2.7. We would need to upgrade the base image to be bookworm. But as this is a heavy breaking change for everybody who is using and extending the 8.2 image, I'm not sure if this should be done.

op-euga commented 9 months ago

In my imagination a 8.2 should always be the newest 8.2 version. if someones needs the the buster version they could use 8.2-buster. in 8.2-alpine the version is bumped to 8.2.14 why not on 8.2?

htuscher commented 9 months ago

We're not responsible for the underlaying PHP version. We just enhance the official PHP images. We've built the image webdevops/php:8.2 based on php:8.2-fpm-buster. So buster is the underlaying Linux distro and changing that while keeping the same image tag would be a major breaking change for a lot of people.

op-euga commented 9 months ago

Maybe it would be a good idea to build webdevops/php:8.2 based on php:8.2-fpm and not php:8.2-fpm-buster because if the underlying distro of php:8.2-fpm changes it would be useful that the webdevops/php:8.2 also changes the distro.

nick-o-dimm commented 7 months ago

We've built the image webdevops/php:8.2 based on php:8.2-fpm-buster. So buster is the underlaying Linux distro and changing that while keeping the same image tag would be a major breaking change for a lot of people.

They no longer update php:8.x-fpm-buster images since June 2023 ( Link to PR ). They added new LTS versions instead (php:8.x-fpm-bookworm). That way webdevops/php:8.2 stuck on PHP v8.2.7 and no longer get any PHP security updates. I completely agree, it would be a breaking change to start using the new Debian LTS version. To resolve this we could probably use new tags for the images, something like webdevops/php-fpm-bookworm:8.2. But I guess it will take a lot of work to adapt everything to the new LTS version :(

AlbinoDrought commented 4 months ago

(Update from buster to bookworm, effectively PHP 8.2.7 to PHP 8.2.20, in 09d1541054d7cfdb885f23550ebfa2a7b485ad5a )