fix: parallelChunkUploads should respect parallelUploads

fix dropzone#2030 based on PR dropzone#2249 by @labor4
NicolasCARPi · Apr 23, 2024 · 1df14d3 · 1df14d3
1 parent babeb39
commit 1df14d3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/dropzone.js b/src/dropzone.js
@@ -1277,7 +1277,12 @@ export default class Dropzone extends Emitter {
         };
 
         if (this.options.parallelChunkUploads) {
-          for (let i = 0; i < file.upload.totalChunkCount; i++) {
+          // we want to limit parallelChunkUploads to the same value as parallelUploads option
+          const parallelCount = Math.min(
+            this.options.parallelChunkUploads === true ? this.options.parallelUploads : this.options.parallelChunkUploads,
+            file.upload.totalChunkCount
+          );
+          for (let i = 0; i < parallelCount; i++) {
             handleNextChunk();
           }
         } else {

diff --git a/src/options.js b/src/options.js
@@ -65,6 +65,7 @@ let defaultOptions = {
 
   /**
    * If `true`, the individual chunks of a file are being uploaded simultaneously.
+   * The limit of concurrent connections is governed by `parallelUploads`.
    */
   parallelChunkUploads: false,