From eb8ce91735af2e174e84c18c6412f757d062d58d Mon Sep 17 00:00:00 2001
From: Jannik Stehle <50302941+JammingBen@users.noreply.github.com>
Date: Fri, 5 May 2023 16:33:10 +0200
Subject: [PATCH] Limit amount of concurrent tus requests (#8987)

---
 changelog/unreleased/bugfix-limit-concurrent-tus-requests | 6 ++++++
 packages/web-runtime/src/services/uppyService.ts          | 4 ++++
 2 files changed, 10 insertions(+)
 create mode 100644 changelog/unreleased/bugfix-limit-concurrent-tus-requests

diff --git a/changelog/unreleased/bugfix-limit-concurrent-tus-requests b/changelog/unreleased/bugfix-limit-concurrent-tus-requests
new file mode 100644
index 00000000000..bce88eebb66
--- /dev/null
+++ b/changelog/unreleased/bugfix-limit-concurrent-tus-requests
@@ -0,0 +1,6 @@
+Bugfix: Limit amount of concurrent tus requests
+
+The amount of concurrent tus requests when uploading has been reduced to 5. This fixes an issue where the access token renewal failed during an ongoing upload because of the sheer amount of pending requests.
+
+https://github.com/owncloud/web/pull/8987
+https://github.com/owncloud/web/issues/8977
diff --git a/packages/web-runtime/src/services/uppyService.ts b/packages/web-runtime/src/services/uppyService.ts
index cb1fdcc19ef..6876a5c68b4 100644
--- a/packages/web-runtime/src/services/uppyService.ts
+++ b/packages/web-runtime/src/services/uppyService.ts
@@ -59,12 +59,16 @@ export class UppyService {
       overridePatchMethod: !!tusHttpMethodOverride,
       retryDelays: [0, 500, 1000],
       uploadDataDuringCreation,
+      limit: 5,
       onBeforeRequest,
       onShouldRetry: (err, retryAttempt, options, next) => {
         // status code 5xx means the upload is gone on the server side
         if (err?.originalResponse?.getStatus() >= 500) {
           return false
         }
+        if (err?.originalResponse?.getStatus() === 401) {
+          return true
+        }
         return next(err)
       }
     }