Changing logic of container logger to color the stderr with red

roy-codefresh · roy-codefresh · commit 48c62d855902 · 2017-02-09T10:47:16.000+02:00
diff --git a/lib/ContainerLogger.js b/lib/ContainerLogger.js
@@ -30,8 +30,6 @@ class ContainerLogger {
                 }
             })
             .then(([stdout, stderr]) => {
-                this._logMessageToFirebase('New logger started v8\n');
-                
                 logger.info(`Attached stream to container: ${this.containerId}`);
                 // Listening on the stream needs to be performed different depending if a tty is attached or not
                 // See documentation of the docker api here: https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#/attach-to-a-container

Original file line number	Diff line number	Diff line change
`@@ -30,8 +30,6 @@ class ContainerLogger {`
`30`	`30`	`}`
`31`	`31`	`})`
`32`	`32`	`.then(([stdout, stderr]) => {`
`33`		`- this._logMessageToFirebase('New logger started v8\n');`
`34`		`-`
`35`	`33`	logger.info(`Attached stream to container: ${this.containerId}`);
`36`	`34`	`// Listening on the stream needs to be performed different depending if a tty is attached or not`
`37`	`35`	`// See documentation of the docker api here: https://docs.docker.com/engine/reference/api/docker_remote_api_v1.24/#/attach-to-a-container`