Skip to content
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

[stable24] Respect user settings in php.ini if they are big enough #33644

Merged
merged 3 commits into from
Aug 23, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

backport of #32216

mickenordin and others added 3 commits August 22, 2022 21:36
In the admin guide:
* https://docs.nextcloud.com/server/latest/admin_manual/configuration_files/big_file_upload_configuration.html

it is mentioned that you can tweek:
* max_input_time
* max_execution_time

in order to enable larger file uploads. However, the current codebase
will hard code these values to one hour, no matter what the user sets in
php.ini.

This patch will allow the user to set these settings in php.ini and they
will be respected, if and only if, they are set to something bigger than
3600 seconds.

Signed-off-by: Micke Nordin <kano@sunet.se>
Co-authored-by: Louis <6653109+artonge@users.noreply.github.com>
Signed-off-by: Micke Nordin <kano@sunet.se>
Signed-off-by: Micke Nordin <kano@sunet.se>
@szaimen szaimen added the 3. to review Waiting for reviews label Aug 22, 2022
@szaimen szaimen requested review from artonge, Pytal, a team, ArtificialOwl, blizzz and juliusknorr and removed request for a team August 22, 2022 21:50
@blizzz blizzz merged commit fcff68a into stable24 Aug 23, 2022
@blizzz blizzz deleted the backport/32216/stable24 branch August 23, 2022 08:29
@blizzz blizzz mentioned this pull request Aug 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants