• ansicolor 0.6.0

      With "ANSI color" plugin turned on, strange symbols are put into the build logs before job names. Like this:

      Started by upstream project "ha:AAAAkR
      +LCAAAAAAAAABb85aBtbiIQSajNKU4P08vOT+vOD8nVc+jsiC1KCczL9svvyT1dMUiOWdZ/
      mImBiZPBrac1Lz0kgwfBubSopwSBiGfrMSyRP2cxLx0/
      eCSosy8dOuKIgYpNOOcITTIMAYIYGRiYKgoADK4SxgE9bPyk/
      Sd3eJLUotL4hNzcvQBD6FtuZYAAAA=test_all" build number ha:AAAAlB
      +LCAAAAAAAAABb85aBtbiIQSajNKU4P08vOT+vOD8nVc+jsiC1KCczL9svvyT1dMUiOWdZ/
      mImBiZPBrac1Lz0kgwfBubSopwSBiGfrMSyRP2cxLx0/
      eCSosy8dOuKIgYpNOOcITTIMAYIYGRiYKgoADKYSxhE9LPyk/
      Sd3eJLUotL4hNzcvTNDIwBh3tcb5kAAAA=603
      Building remotely on slave4

          [JENKINS-11752] ANSI color plugin adds garbage to log

          Daniel Doubrovkine added a comment - Fixed in https://github.com/dblock/jenkins-ansicolor-plugin/issues/3 .

          In versions 0.3.0 and 0.3.1 of ANSI color plugin the strange symbols show up again. Version 0.2.1 is still ok.

          Hans-Juergen Hafner added a comment - In versions 0.3.0 and 0.3.1 of ANSI color plugin the strange symbols show up again. Version 0.2.1 is still ok.

          Patrick Byrne added a comment -

          This affects us at the moment.

          Jenkins ver 1.508
          ANSIColor 0.31

          It makes our log files 1000x larger - this is ridiculous!

          Patrick Byrne added a comment - This affects us at the moment. Jenkins ver 1.508 ANSIColor 0.31 It makes our log files 1000x larger - this is ridiculous!

          Anze Zagar added a comment -

          Suggested temporary fix (until JENKINS-13816 allows better way of doing this):

          — src/main/java/hudson/plugins/ansicolor/AnsiColorNote.java (revision 36259)
          +++ src/main/java/hudson/plugins/ansicolor/AnsiColorNote.java (working copy)
          @@ -65,6 +65,7 @@
          */
          @Override
          public ConsoleAnnotator annotate(Object context, MarkupText text, int charPos) {
          + if (this.data.contains(ConsoleNote.PREAMBLE_STR)) return null;
          try {
          String colorizedData = colorize(StringEscapeUtils.escapeHtml(this.data), this.getColorMap());
          if (! colorizedData.contentEquals(this.data)) {

          Anze Zagar added a comment - Suggested temporary fix (until JENKINS-13816 allows better way of doing this): — src/main/java/hudson/plugins/ansicolor/AnsiColorNote.java (revision 36259) +++ src/main/java/hudson/plugins/ansicolor/AnsiColorNote.java (working copy) @@ -65,6 +65,7 @@ */ @Override public ConsoleAnnotator annotate(Object context, MarkupText text, int charPos) { + if (this.data.contains(ConsoleNote.PREAMBLE_STR)) return null; try { String colorizedData = colorize(StringEscapeUtils.escapeHtml(this.data), this.getColorMap()); if (! colorizedData.contentEquals(this.data)) {

          This also increases the log sizes considerably. 100k -> 16mb

          Jan Vansteenkiste added a comment - This also increases the log sizes considerably. 100k -> 16mb

          Still true for ansicolor 0.4.1 :/

          Jan Vansteenkiste added a comment - Still true for ansicolor 0.4.1 :/

          Patrick Byrne added a comment -

          Yes, these appear to be ~2Kb (7-zip) compressed files embedded in each line of output. This has to be one of the most egregious examples of 'bloat' which I have seen. I would love to hear an explanation of why this was ever thought to be a good idea.

          Patrick Byrne added a comment - Yes, these appear to be ~2Kb (7-zip) compressed files embedded in each line of output . This has to be one of the most egregious examples of 'bloat' which I have seen. I would love to hear an explanation of why this was ever thought to be a good idea.

          I updated the prio on this to major because nodejs projects tend to have MASSIVE logs when this is enabled.

          Jan Vansteenkiste added a comment - I updated the prio on this to major because nodejs projects tend to have MASSIVE logs when this is enabled.

          Is there any movement on this? Could the ansi information be stored some other way? Or re-calculated on the fly when displayed?

          Christian Höltje added a comment - Is there any movement on this? Could the ansi information be stored some other way? Or re-calculated on the fly when displayed?

          Devin Nusbaum added a comment -

          Ansicolor 0.6.0 was just released with a fix for this issue.

          Devin Nusbaum added a comment - Ansicolor 0.6.0 was just released with a fix for this issue.

            jglick Jesse Glick
            alex01ves Alex Vesely
            Votes:
            11 Vote for this issue
            Watchers:
            15 Start watching this issue

              Created:
              Updated:
              Resolved: