Add newline to pipeline stdin for POSIX compliance

See #568
sbs20 · Feb 28, 2023 · cb27b23 · cb27b23
1 parent 6f1be02
commit cb27b23
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/packages/server/src/scan-controller.js b/packages/server/src/scan-controller.js
@@ -83,13 +83,13 @@ class ScanController {
 
     // Apply filters
     if (this.request.filters.length > 0) {
-      const stdin = files.map(f => f.name).join('\n');
+      const stdin = files.map(f => `${f.name}\n`).join('');
       const cmd = `convert @- ${application.filterBuilder().build(this.request.filters)} f-%04d.tif`;
       await Process.spawn(cmd, stdin, { cwd: config.tempDirectory });
       files = (await this.listFiles()).filter(f => f.name.match(/f-\d{4}\.tif/));
     }
 
-    const stdin = files.map(f => f.name).join('\n');
+    const stdin = files.map(f => `${f.name}\n`).join('');
     log.debug('Executing cmds:', this.pipeline.commands);
     const stdout = await Process.chain(this.pipeline.commands, stdin, { cwd: config.tempDirectory });
     let filenames = stdout.toString().split('\n').filter(f => f.length > 0);