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

PHP unit ignores PHP_UPLOAD_MAX_FILE_SIZE #394

Closed lenvanessen closed 1 month ago

lenvanessen commented 1 month ago

Steps To Reproduce

  1. Build a container using the serversideup/php:8.2-unit
  2. create a script with a phpinfo(); dump
  3. upload_max_filesize is unset

Outcome

What did you expect?

For upload_max_filesize to be set to the 100m default

What happened instead?

upload_max_filesize is undefined

Affected Docker Images

Anything else?

No response

jaydrogers commented 1 month ago

Thanks for pointing this out! I accidentally had PHP_UPLOAD_MAX_FILESIZEinstead of PHP_UPLOAD_MAX_FILE_SIZE in the PHP ini.

It worked in PHP-FPM (only on web) because it was set correctly in the FPM config. This will be fixed in v3.3.0. Thanks for your help!