fix: multiples should only affect multipart and urlencoded

node-formidable · Jan 23, 2021 · d33884e · d33884e
1 parent 5d4dc40
commit d33884e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Formidable.js b/src/Formidable.js
@@ -142,7 +142,9 @@ class IncomingForm extends EventEmitter {
       const files = {};
 
       this.on('field', (name, value) => {
-        if (this.options.multiples) {
+        if (this.options.multiples && 
+            (this.type === 'multipart' || this.type === 'urlencoded')
+            ) {
           const mObj = { [name]: value };
           mockFields = `${mockFields}&${qs.stringify(mObj)}`;
         } else {

diff --git a/test/integration/test-json.js b/test/integration/test-json.js
@@ -11,7 +11,7 @@ const testData = {
 
 const PORT = 13532;
 const server = http.createServer((req, res) => {
-  const form = formidable();
+  const form = formidable({multiples: true});
 
   form.parse(req, (err, fields) => {
     assert.deepStrictEqual(fields, {