Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-11752

ANSI color plugin adds garbage to log

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      ansicolor 0.6.0

      Description

      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

        Attachments

          Issue Links

            Activity

            alex01ves Alex Vesely created issue -
            Hide
            cweiske_nr Christian Weiske added a comment -

            Same problem here:

            With ANSI color

            ha:AAAAZB+LCAAAAAAAAABb85aBtbiIQSejNKU4P0+vIKc0PTOvWK8gIzMvXS85P684PydVLwDEC0ksSk8t8csvSe1Zynx/wmzdSUwMDBVFDFJQrTDFzhAapJABAhhBCgsAxIr4amoAAAA=nr_semantic_annotations > prepare:
            
            ha:AAAAZB+LCAAAAAAAAABb85aBtbiIQSejNKU4P0+vIKc0PTOvWK8gIzMvXS85P684PydVLwDEC0ksSk8t8csvSe1Zynx/wmzdSUwMDBVFDFJQrTDFzhAapJABAhhBCgsAxIr4amoAAAA=nr_semantic_annotations > pdepend:
            

            Without ANSI color

            nr_semantic_annotations > prepare:
            
            nr_semantic_annotations > pdepend:
            
            Show
            cweiske_nr Christian Weiske added a comment - Same problem here: With ANSI color ha:AAAAZB+LCAAAAAAAAABb85aBtbiIQSejNKU4P0+vIKc0PTOvWK8gIzMvXS85P684PydVLwDEC0ksSk8t8csvSe1Zynx/wmzdSUwMDBVFDFJQrTDFzhAapJABAhhBCgsAxIr4amoAAAA=nr_semantic_annotations > prepare: ha:AAAAZB+LCAAAAAAAAABb85aBtbiIQSejNKU4P0+vIKc0PTOvWK8gIzMvXS85P684PydVLwDEC0ksSk8t8csvSe1Zynx/wmzdSUwMDBVFDFJQrTDFzhAapJABAhhBCgsAxIr4amoAAAA=nr_semantic_annotations > pdepend: Without ANSI color nr_semantic_annotations > prepare: nr_semantic_annotations > pdepend:
            Show
            dblock Daniel Doubrovkine added a comment - Fixed in https://github.com/dblock/jenkins-ansicolor-plugin/issues/3 .
            dblock Daniel Doubrovkine made changes -
            Field Original Value New Value
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Resolved [ 5 ]
            Hide
            hjhafner 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.

            Show
            hjhafner 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.
            hjhafner Hans-Juergen Hafner made changes -
            Resolution Fixed [ 1 ]
            Status Resolved [ 5 ] Reopened [ 4 ]
            Hide
            pjlbyrne 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!

            Show
            pjlbyrne 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!
            Hide
            anzez 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)) {

            Show
            anzez 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)) {
            Hide
            vstone Jan Vansteenkiste added a comment -

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

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

            Still true for ansicolor 0.4.1 :/

            Show
            vstone Jan Vansteenkiste added a comment - Still true for ansicolor 0.4.1 :/
            Hide
            pjlbyrne 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.

            Show
            pjlbyrne 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.
            vstone Jan Vansteenkiste made changes -
            Priority Minor [ 4 ] Major [ 3 ]
            Hide
            vstone 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.

            Show
            vstone 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.
            rtyler R. Tyler Croy made changes -
            Workflow JNJira [ 141995 ] JNJira + In-Review [ 186026 ]
            Hide
            docwhat 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?

            Show
            docwhat 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?
            jglick Jesse Glick made changes -
            Link This issue is duplicated by JENKINS-29090 [ JENKINS-29090 ]
            jglick Jesse Glick made changes -
            Remote Link This issue links to "PR 132 (Web Link)" [ 22006 ]
            jglick Jesse Glick made changes -
            Assignee Jesse Glick [ jglick ]
            jglick Jesse Glick made changes -
            Status Reopened [ 4 ] Open [ 1 ]
            jglick Jesse Glick made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            jglick Jesse Glick made changes -
            Status In Progress [ 3 ] In Review [ 10005 ]
            jglick Jesse Glick made changes -
            Status In Review [ 10005 ] In Progress [ 3 ]
            jglick Jesse Glick made changes -
            Status In Progress [ 3 ] Open [ 1 ]
            jglick Jesse Glick made changes -
            Resolution Fixed [ 1 ]
            Status Open [ 1 ] Fixed but Unreleased [ 10203 ]
            Hide
            dnusbaum Devin Nusbaum added a comment -

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

            Show
            dnusbaum Devin Nusbaum added a comment - Ansicolor 0.6.0 was just released with a fix for this issue.
            dnusbaum Devin Nusbaum made changes -
            Released As ansicolor 0.6.0
            Status Fixed but Unreleased [ 10203 ] Resolved [ 5 ]

              People

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

                Dates

                Created:
                Updated:
                Resolved: