-
Notifications
You must be signed in to change notification settings - Fork 7.9k
post_max_size evaluates .user.ini too late in php-fpm? #8157
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
Comments
That might be caused by |
This issue does not affect (F)CGI, so this would be an FPM only issue; however, I'm not sure what exactly is wrong, since the docs state;
Either the docs are wrong (should mention FPM as well), and the setting in .user.ini should work, or the docs are right, and .user.ini should be ignored altogether by FPM. |
This is basically the same as https://bugs.php.net/bug.php?id=75741 . The ordering will be most likely the issue. I have got this on my list but might take few months to get there as there are higher priority items. |
Uh oh!
There was an error while loading. Please reload this page.
Description
We are trying to modify the value for
post_max_size
per directory through a.user.ini
-file running on php-fpm behind an Apache webserver using proxy_fgci_module. As mentioned in the docs, the setting should be a per-dir setting and we are able to see the correct values using$value = ini_get('post_max_size');
but sending a bigger value using the POST method results in a log message using the default value of 8M and the data is not submitted to the script:By changing the value in the php.ini, it's working as expected. Also using PUT instead of POST let me pass and receive the data.
I'd guess, the POST content size is evaluated too early, before the value of the .user.ini is set?
The
upload_max_filesize
value is working fine and we are able to upload bigger files than the limit, only POST-requests with content bigger than 8M are affected.PHP Version
Tested on 7.4 and 8.1
Operating System
No response
The text was updated successfully, but these errors were encountered: