coop-cloud/nextcloud
9
5
Fork 11
Code Issues 18 Pull Requests 11 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 4 Commits 1 Files Changed 3
+8 -2
p4u1 commented 2026-06-03 16:23:51 +00:00
Owner
Copy Link
Copy Source
No description provided.
p4u1 added 1 commit 2026-06-03 16:23:52 +00:00
fix: ensure large uploads work
continuous-integration/drone/pr Build is failing
Details
394b68a920
fauno commented 2026-06-03 16:42:49 +00:00
Owner
Copy Link
Copy Source

@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
Copy Source

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
Copy Source

@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
fauno commented 2026-06-28 22:40:44 +00:00
Owner
Copy Link
Copy Source

i'm reading traefik's docs and apparently the maxRequestBodyBytes option is already 0 by default, but memRequestBodyBytes is set at 1MB, should we increase this?

i'm reading traefik's docs and apparently the maxRequestBodyBytes option is already 0 by default, but [memRequestBodyBytes](https://doc.traefik.io/traefik/reference/routing-configuration/http/middlewares/buffering/#opt-memRequestBodyBytes) is set at 1MB, should we increase this?
Some checks are pending
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 (Aadil Ayub) abra-bot (Abra Bot) ammaratef45 amras (Sarma) Apfelwurm appletalk arjan basebuilder BornDeleuze Brooke carla cas (Cassowary) codegod100 coopcloud cyrnel decentral1se (decentral1se) dede devydave fauno (fauno) flancian Frando iexos jade (Jade Ambrose) javielico (Javielico) jjsfunhouse jmakdah2 joe-irving (Joe Irving) kawaiipunk (KawaiiPunk) knoflook kolaente lambdabundesverband linnealovespie marlon (marlon) mayel mirsal moosemower moritz nicksellen notplants oxaliq (sorrel) p4u1 pau pharaohgraphy (Andrew 🐦‍🔥❤️‍🔥✴️) PhiNatalie renovate-bot (Comrade Renovate Bot) ripclap rix rscmbbng sef (sef) simon sixsmith (Sixsmith) stevensting tobias trav val (val (he/him)) vaznasty virtualboys wolcen (Chris Thompson) wykwit xynosis yksflip
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coop-cloud/nextcloud#84
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?