Skip to content

Commit 45317f5

Browse files
Updated the task-logger version (#78) (#83)
1 parent f55c676 commit 45317f5

File tree

2 files changed

+18
-5
lines changed

2 files changed

+18
-5
lines changed

lib/ContainerLogger.js

+17-4
Original file line numberDiff line numberDiff line change
@@ -102,14 +102,21 @@ class ContainerLogger extends EventEmitter {
102102
logger.info(`Piping stdout and stderr step streams`);
103103

104104
const stepLoggerWritableStream = this.stepLogger.writeStream();
105+
const { useLogsTimestamps } = this.stepLogger.opts;
105106
stepLoggerWritableStream.on('error', (err) => logger.error(`stepLoggerWritableStream: ${err}`));
106107

107108
// Attention(!) all streams piped to step logger writable stream must be a new streams(!) in order to avoid message piping twice to writable stream.
108109
// { end = false } on the stepLoggerWritableStream because there is only one instance of it for all the steps.
109110
this.handledStreams++;
110-
stdout
111+
let stdoutStream = stdout
111112
.pipe(this._logSizeLimitStream())
112-
.pipe(this.stepLogger.createMaskingStream())
113+
.pipe(this.stepLogger.createMaskingStream());
114+
115+
if (useLogsTimestamps) {
116+
stdoutStream = stdoutStream.pipe(this.stepLogger.createPrependTimestampsStream());
117+
}
118+
119+
stdoutStream
113120
.pipe(this.stepLogger.stepNameTransformStream().once('end', this._handleFinished.bind(this)))
114121
.pipe(stepLoggerWritableStream, { end: false });
115122

@@ -118,10 +125,16 @@ class ContainerLogger extends EventEmitter {
118125
}
119126

120127
this.handledStreams++;
121-
stderr
128+
let stderrStream = stderr
122129
.pipe(this._logSizeLimitStream())
123130
.pipe(this._errorTransformerStream())
124-
.pipe(this.stepLogger.createMaskingStream())
131+
.pipe(this.stepLogger.createMaskingStream());
132+
133+
if (useLogsTimestamps) {
134+
stderrStream = stderrStream.pipe(this.stepLogger.createPrependTimestampsStream());
135+
}
136+
137+
stderrStream
125138
.pipe(this.stepLogger.stepNameTransformStream().once('end', this._handleFinished.bind(this)))
126139
.pipe(stepLoggerWritableStream, { end: false });
127140

service.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
version: 1.5.9
1+
version: 1.6.0

0 commit comments

Comments
 (0)