Properly synchronize access so we do not risk having a timestamp in t…

…he middle of a line.
jenkinsci · Oct 25, 2018 · bf1e61c · bf1e61c
1 parent 9b19c6e
commit bf1e61c
Showing 1 changed file with 7 additions and 5 deletions.
diff --git a/src/main/java/hudson/plugins/timestamper/pipeline/GlobalDecorator.java b/src/main/java/hudson/plugins/timestamper/pipeline/GlobalDecorator.java
@@ -66,11 +66,13 @@ public OutputStream decorate(final OutputStream logger) throws IOException, Inte
         return new LineTransformationOutputStream() {
             @Override
             protected void eol(byte[] b, int len) throws IOException {
-                logger.write('[');
-                logger.write(f.format(new Date()).getBytes(StandardCharsets.US_ASCII));
-                logger.write(']');
-                logger.write(' ');
-                logger.write(b, 0, len);
+                synchronized (logger) { // typically this will be a PrintStream
+                    logger.write('[');
+                    logger.write(f.format(new Date()).getBytes(StandardCharsets.US_ASCII));
+                    logger.write(']');
+                    logger.write(' ');
+                    logger.write(b, 0, len);
+                }
             }
         };
     }