coop-cloud/nextcloud
9
5
Fork 11
Code Issues 18 Pull Requests 9 Releases Wiki Activity

WIP: fix: ensure large uploads work #84

Draft
p4u1 wants to merge 1 commits from upload-limit into main
pull from: upload-limit
merge into: coop-cloud:main
Conversation 3 Commits 1 Files Changed 3 +8 -2
p4u1 commented 2026-06-03 16:23:51 +00:00
Owner
Copy Link
No description provided.
p4u1 added 1 commit 2026-06-03 16:23:52 +00:00
fix: ensure large uploads work
Some checks failed
continuous-integration/drone/pr Build is failing
Details
394b68a920
fauno commented 2026-06-03 16:42:49 +00:00
Owner
Copy Link

@p4u1 is this ready for testing? we're having this same issue

@p4u1 is this ready for testing? we're having this same issue
fauno commented 2026-06-03 18:51:32 +00:00
Owner
Copy Link

we tested it anyway! this is what we observe, taking into account we're reverse-proxying coopcloud's traefik with an nginx server (with really high client body size and timeout limits):

  • PHP_UPLOAD_LIMIT=20G
  • with a 3GB file, the web client creates 5 chunks and starts 5 PUT requests
  • the first request fails with 504 gateway timeout error
  • web client cancels the other 4 with 499 client closed connection
  • abra logs 400 errors for the 5 requests on recipe's nginx
  • no php logs (i've never seen a php-fpm log that worked btw)
we tested it anyway! this is what we observe, taking into account we're reverse-proxying coopcloud's traefik with an nginx server (with really high client body size and timeout limits): * `PHP_UPLOAD_LIMIT=20G` * with a 3GB file, the web client creates 5 chunks and starts 5 PUT requests * the first request fails with 504 gateway timeout error * web client cancels the other 4 with 499 client closed connection * abra logs 400 errors for the 5 requests on recipe's nginx * no php logs (i've never seen a php-fpm log that worked btw)
p4u1 commented 2026-06-05 07:36:25 +00:00
Author
Owner
Copy Link

@fauno for me this worked for a 4gb upload. But I also needed the traefik fix coop-cloud/traefik#107

I don't really have time right now to bring both PRS over the finish line so feel free to continue

@fauno for me this worked for a 4gb upload. But I also needed the traefik fix https://git.coopcloud.tech/coop-cloud/traefik/pulls/107 I don't really have time right now to bring both PRS over the finish line so feel free to continue
Some checks failed
continuous-integration/drone/pr Build is failing
Details
This pull request is marked as a work in progress.
This branch is out-of-date with the base branch
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin upload-limit:upload-limit
git checkout upload-limit
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
documentation
For issues that are calling out documentation gap
duplicate
This issue or pull request already exists
enhancement
New feature
help wanted
Need some help
invalid
Something is wrong
question
More information is needed
wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
3wordchant aadil abra-bot ammaratef45 amras Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas codegod100 coopcloud cyrnel decentral1se dede devydave fauno flancian Frando iexos jade javielico jjsfunhouse jmakdah2 joe-irving kawaiipunk knoflook kolaente lambdabundesverband linnealovespie marlon mayel mirsal moosemower moritz nicksellen notplants oxaliq p4u1 pau pharaohgraphy PhiNatalie renovate-bot ripclap rix rscmbbng sef simon sixsmith stevensting tobias trav val vaznasty virtualboys wolcen wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#84
No description provided.
Delete Branch "upload-limit"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?