Merge pull request #437 from Inist-CNRS/fix-csv-object

fix: 🐛 avoid crash with bad data
Inist-CNRS · Dec 20, 2024 · 33ca88c · 33ca88c
2 parents f7f3cee + 0a7e997
commit 33ca88c
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/packages/basics/src/csv-object.js b/packages/basics/src/csv-object.js
@@ -4,7 +4,7 @@ function CSVObject(data, feed) {
     }
 
     if (!this.columns) {
-        this.columns = data.map((name) => name.replace(/\./g, ''));
+        this.columns = Array().concat(data).map((name) => String(name).replace(/\./g, ''));
 
         const countCols = {};
         const renamedCols = {};

diff --git a/packages/basics/test/basics.js b/packages/basics/test/basics.js
@@ -122,6 +122,25 @@ describe('test', () => {
                 done();
             });
     });
+    it('CSVObject Error', (done) => {
+        const res = [];
+        from([
+            true,
+            'vrai',
+            'ok'
+        ])
+            .pipe(ezs('CSVObject'))
+            .pipe(ezs.catch())
+            .on('error', done)
+            .on('data', (chunk) => {
+                res.push(chunk);
+            })
+            .on('end', () => {
+                assert.equal(0, res.length);
+                done();
+            });
+    });
+
 
     it('JSONString #0a', (done) => {
         let res = '';