Skip to content

[stable22] Respect user settings in php.ini if they are big enough - #33646

Closed
backportbot-nextcloud[bot] wants to merge 3 commits into
stable22from
backport/32216/stable22
Closed

[stable22] Respect user settings in php.ini if they are big enough#33646
backportbot-nextcloud[bot] wants to merge 3 commits into
stable22from
backport/32216/stable22

Conversation

@backportbot-nextcloud

@backportbot-nextcloudbackportbot-nextcloudBot commented Aug 22, 2022

Copy link
Copy Markdown
  • ⚠️ This backport had conflicts and is incomplete

backport of #32216

mickenordinand others added 3 commits August 22, 2022 22:00
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

Copy link
Copy Markdown
Contributor

22 is EOL

@szaimenszaimen closed this Aug 22, 2022
@szaimenszaimen removed this from the Nextcloud 22.2.11 milestone Aug 22, 2022
@szaimen
szaimen deleted the backport/32216/stable22 branch August 22, 2022 22:02
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@szaimen@mickenordin