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

Fix uploading large files #482

Open
wants to merge 3 commits into
base: previous
Choose a base branch
from
Open

Conversation

quietsy
Copy link
Member

@quietsy quietsy commented Oct 25, 2024

Fix uploading large files which is a very common support issue

@quietsy quietsy requested a review from a team October 25, 2024 13:47
@LinuxServer-CI
Copy link
Collaborator

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-7fa5bf48030f36fc3328f30164dee85617cee03c-pr-482/index.html
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-7fa5bf48030f36fc3328f30164dee85617cee03c-pr-482/shellcheck-result.xml

Tag Passed
amd64-previous-29.0.8-pkg-ddebef85-dev-7fa5bf48030f36fc3328f30164dee85617cee03c-pr-482
arm64v8-previous-29.0.8-pkg-ddebef85-dev-7fa5bf48030f36fc3328f30164dee85617cee03c-pr-482

@LinuxServer-CI
Copy link
Collaborator

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-5d95254a7f8ee99078325b9e332393110d2c5dca-pr-482/index.html
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-5d95254a7f8ee99078325b9e332393110d2c5dca-pr-482/shellcheck-result.xml

Tag Passed
amd64-previous-29.0.8-pkg-ddebef85-dev-5d95254a7f8ee99078325b9e332393110d2c5dca-pr-482
arm64v8-previous-29.0.8-pkg-ddebef85-dev-5d95254a7f8ee99078325b9e332393110d2c5dca-pr-482

@LinuxServer-CI
Copy link
Collaborator

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-e57cfdf8150b809da502b7b109d11fd551d0188e-pr-482/index.html
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-ddebef85-dev-e57cfdf8150b809da502b7b109d11fd551d0188e-pr-482/shellcheck-result.xml

Tag Passed
amd64-previous-29.0.8-pkg-ddebef85-dev-e57cfdf8150b809da502b7b109d11fd551d0188e-pr-482
arm64v8-previous-29.0.8-pkg-ddebef85-dev-e57cfdf8150b809da502b7b109d11fd551d0188e-pr-482

@LinuxServer-CI
Copy link
Collaborator

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-7f93e87a-dev-d2dd309fe8501f76f424a04f2254f6670b9f3290-pr-482/index.html
https://ci-tests.linuxserver.io/lspipepr/nextcloud/previous-29.0.8-pkg-7f93e87a-dev-d2dd309fe8501f76f424a04f2254f6670b9f3290-pr-482/shellcheck-result.xml

Tag Passed
amd64-previous-29.0.8-pkg-7f93e87a-dev-d2dd309fe8501f76f424a04f2254f6670b9f3290-pr-482
arm64v8-previous-29.0.8-pkg-7f93e87a-dev-d2dd309fe8501f76f424a04f2254f6670b9f3290-pr-482

@mmartinortiz
Copy link

Any chance that this MR gets completed? There are another two MR with the same title and similar changes: #481 #480

@drizuid
Copy link
Member

drizuid commented Dec 9, 2024

Any chance that this MR gets completed? There are another two MR with the same title and similar changes: #481 #480

eventually, but nothing stops any user from following the nextcloud documentation (which is what was done) to accomplish this.

@mmartinortiz
Copy link

eventually, but nothing stops any user from following the nextcloud documentation (which is what was done) to accomplish this.

If I read the documentation correctly, besides the client_max_body_size nginx parameter, we also have to change values in the php.ini file (such values are modified by this MR, in the nextcloud.ini file)

I've modified the file under php/php-local.ini as it is the only one mounted in a volume. Is this the right file to modify and get the same effect that the configuration changes of this PR?

In my case, I'm getting a "HTTPError: Response code 413 (Request Entity Too Large))" when I upload files. The same files can be uploaded via the web interface, but I read that they are uploaded in smaller chunks to avoid this issue.

@nemchik
Copy link
Member

nemchik commented Dec 10, 2024

I've modified the file under php/php-local.ini as it is the only one mounted in a volume. Is this the right file to modify and get the same effect that the configuration changes of this PR?

Yes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: PRs Approved
Development

Successfully merging this pull request may close these issues.

5 participants