-
Notifications
You must be signed in to change notification settings - Fork 2k
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
php8.2.15 removed ftp extension #1488
Comments
Hmm, I think we need to dig into the "Before PHP 7.0.0" bit of https://www.php.net/manual/en/function.ftp-ssl-connect.php, because from the docs, this should be working fine. 😕 |
Ah, this is due to https://github.com/php/php-src/blob/php-8.2.15/ext/ftp/config.m4#L21-L24, but just passing |
Confirmed, installing |
More concretely: FROM php:8.2
RUN apt-get update && apt-get install -y libssl-dev
RUN docker-php-ext-configure ftp --with-openssl-dir=/usr \
&& docker-php-ext-install ftp |
Will the ftp be added back in the future Docker images? I need to know it to decide whether to apply manual ftp installation temporarily or permanently. Thanks. |
Unfortunately, we don't plan to add it back to the default set, no. |
@tianon thanks for the workaround and clarifying that is not be added back. Is there any reason that got the package removed in the first place in a minor upgrade? Thanks |
It should've only been included in the 5.x versions in the original PR that added it (because the underlying issue was fixed in 7+) 😅 The other option we considered was to remove it in the previous version as a normal image update (without waiting for a version change), and we opted for the slightly more conservative switch instead. |
FWIW: I've intentionally listed the necessary steps in #1482, so that it is not necessary for someone else to research this 😅 |
Do you mind to explain how to use this solution? From what I understand I'm supposed to create Dockerfile with from/run commands. Where do I place Dockerfile and how do I call it from my Docker compose file? Can those commands be run from Ubuntu terminal and how? |
When attempting this not with the php8.2 image directly but with a downstream image built on it I run into this error:
However, running |
Hello!
The update from 8.2.14 to 8.2.15 removed the ftp extension (see 7512769), extension that needs to be statically compiled (see #236)
I believe that should not be removed in a minor upgrade. 😢
We fixed to the 8.2.14 but we would like to have minor upgrades without having to manage images from scratch.
Thanks
The text was updated successfully, but these errors were encountered: