-
Notifications
You must be signed in to change notification settings - Fork 54
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
php74-fpm and pcre error with composer if you install any extension #44
Comments
The problem seems related to oerdnj/deb.sury.org#1682 |
its related to all new php updates which rely on a PCRE update / bugfix. Our only way to fix this was to downgrade in the container. The image has nothing to do with this issue because its loading just the latest minor version of php.
choose an older version (for example php7.4 | 7.4.3-4ubuntu2.7)
|
Thanks for the report, I was just looking into it. It indeed seems like an issue with php 7.4.26. The current build (last night) of our image is on 7.4.25 and .26 must've come out in the last few hours. |
8.0.13, 7.4.26 and 7.3.33 are affected |
@thisisablock I can't reproduce it on 7.3.33, but indeed on the others. |
@thisisablock your link didn't work for me - I guess https://bugs.php.net/bug.php?id=81424 is the relevant issue. btw apparently adding a |
Ah, looks like there's also a new version if |
@luispabon can confirm - 7.3.33 is safe |
Unfortunately the new build won't fix the problem as it stands as libpcre isn't being updated. I reckon and I've said as much on the ondrej repo's issue above that the php packages should directly depend on this newer version of pcre (provided by the ppa). I'll work in a workaround. |
Issues: * #44 * oerdnj/deb.sury.org#1682 Closes #44
Workaround is in place, builds in progress: https://github.com/phpdocker-io/base-images/actions/runs/1481517715 Apologies for the slow builds, I recently added arm32 and 64 builds which take some time. |
Builds are up, they're working for me. If any of you could please |
thanks luis! have a nice weekend |
Thanks @luispabon , It is working for me now. (7.4) |
8.0 also working for me now. |
I also can confirm 7.4 works. I'll think to keep in any case the dist-upgrade just in case. |
Thanks all for double checking. Closing this now @tannoiser I personally avoid upgrades on containers because it makes builds non-reproducible, and due to the way layering works on images, container size can balloon. This is part of the reason why images are being built daily, instead of waiting for a new php release. In any case it's not inherently wrong - you do you. |
Thanks for the quick response @luispabon. This leads me to ask you if there's a way to lock the container tag to prevent these kind of issues, so we can manually control the update of PHP / dependencies, giving more security on build status on this side. I went to https://hub.docker.com/r/phpdockerio/php74-fpm/tags and it only shows "latest" |
Indeed there's only latest and it is like so by design. Static tags wouldn't have helped on this issue anyhow, unless I'd also gone and installed into the base image all of the php extension packages for that version of php. And I totally don't want to do that as the images would balloon in size. |
Hi,
starting from today november 19th, php74-fpm image throw an error in preg_match_all() function.
Step to reproduce:
docker run --name fpmtest phpdockerio/php74-fpm
the output error is:
The text was updated successfully, but these errors were encountered: