strange behavior: stream could be a string ! see apocas/dockerode#456

Inist-CNRS · Jun 12, 2018 · 18754e7 · 18754e7 · kerphi · Jun 12, 2018
1 parent 7c20709
commit 18754e7
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/ezmaster-api/routes/v1-instances.js b/ezmaster-api/routes/v1-instances.js
@@ -687,12 +687,19 @@ router
       if (err) {
         return res.status(err.statusCode).send(err.reason).end();
       }
-      stream.on('data', (chunk) => {
-        res.write(stripAnsi(chunk.toString()));
-      });
-      stream.on('end', () => {
+      // strange behavior: stream could be a string !
+      // see https://github.com/apocas/dockerode/issues/456
+      if (typeof(stream) === 'string') {
+        res.write(stripAnsi(stream));
         res.end();
-      });
+      } else {
+        stream.on('data', (chunk) => {
+          res.write(stripAnsi(chunk.toString()));
+        });
+        stream.on('end', () => {
+          res.end();
+        });
+      }
     });
   });
 });