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

Console log contains junk in Hudson 1.352

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Component/s: core
    • Labels:
      None
    • Similar Issues:
      Show 5 results

      Description

      After the fix of JENKINS-5941, /console looks fine but /consoleText and /progressiveLog often contain junk characters. For example,

      http://bertram.netbeans.org/hudson/job/cnd-main/3077/console

      looks fine as far as I could see but

      http://bertram.netbeans.org/hudson/job/cnd-main/3077/consoleText

      contains a bunch of junk characters, as did the console displayed in NetBeans, e.g. (some long lines broken up to display better):

      [8mha:AAAAYB+LCAAAAAAAAABb85aBtbiIQT6jNKU4P0+vJLE4u1gvPjGvRM8xry
      QksSg9tcQvvyTVvqNr/s+Vl9uYGBgqihikoKqT8
      /OK83NS9ZwhNEghAwQwghQWAADROOumXQAAAA==[0mmodule-auto-deps:
           [copy] Copying 1 file to /space/hudson/workspace/cnd-main/nbbuild/netbeans/java/config/ModuleAutoDeps
      
      [8mha:AAAAYB+LCAAAAAAAAABb85aBtbiIQT6jNKU4P0+vJLE4u1gvPjGvRM8xry
      QksSg9tcQvvyTVvqNr/s+Vl9uYGBgqihikoKqT8/OK83NS9ZwhNEghAwQwghQWAADROOumXQAAAA==[0mprojectized-common.release:
      

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Indeed this is not fixed in 1.354, e.g.

            http://deadlock.netbeans.org/hudson/job/ruby/3483/consoleText
            
            ...
            all-api.visual:
            Building api.visual...
            
            [8mha:AAAAYB+LCAA.....mXQAAAA==[0mapi.visual.default.init:
            Updating property file: /hudson/workdir/jobs/ruby/workspace/nbbuild/netbeans/moduleCluster.properties
            ...
            
            Show
            jglick Jesse Glick added a comment - Indeed this is not fixed in 1.354, e.g. http://deadlock.netbeans.org/hudson/job/ruby/3483/consoleText ... all-api.visual: Building api.visual... [8mha:AAAAYB+LCAA.....mXQAAAA==[0mapi.visual.default.init: Updating property file: /hudson/workdir/jobs/ruby/workspace/nbbuild/netbeans/moduleCluster.properties ...
            Hide
            starfury starfury added a comment -

            yes, I can confirm that also. garbage still existing.

            Show
            starfury starfury added a comment - yes, I can confirm that also. garbage still existing.
            Hide
            voorth voorth added a comment -

            Still present in 1.355...

            Show
            voorth voorth added a comment - Still present in 1.355...
            Hide
            kalpanab kalpanab added a comment -

            Hi All,

            I too had same issue with 1.362 version. I am not sure weather it is right fix or not but I could able to fix it by changing the code listed below in Run.java file. See the code change below if it helps to you.

            In Run.java class method changed is doConsoleText(StaplerRequest req, StaplerResponse rsp)

            Comment below lines:

            // Prevent jelly from flushing stream so Content-Length header can be added afterwards
            FlushProofOutputStream out = new FlushProofOutputStream(rsp.getCompressedOutputStream(req));
            getLogText().writeLogTo(0,out);
            out.close();

            ***********************************************
            Then add below lines:

            Writer w = rsp.getCompressedWriter(req);
            getLogText().writeHtmlTo(0,w);
            w.close();

            ************************************************

            Show
            kalpanab kalpanab added a comment - Hi All, I too had same issue with 1.362 version. I am not sure weather it is right fix or not but I could able to fix it by changing the code listed below in Run.java file. See the code change below if it helps to you. In Run.java class method changed is doConsoleText(StaplerRequest req, StaplerResponse rsp) Comment below lines: // Prevent jelly from flushing stream so Content-Length header can be added afterwards FlushProofOutputStream out = new FlushProofOutputStream(rsp.getCompressedOutputStream(req)); getLogText().writeLogTo(0,out); out.close(); *********************************************** Then add below lines: Writer w = rsp.getCompressedWriter(req); getLogText().writeHtmlTo(0,w); w.close(); ************************************************
            Hide
            scm_issue_link SCM/JIRA link daemon added a comment -

            Code changed in hudson
            User: : kohsuke
            Path:
            trunk/hudson/main/core/src/main/java/hudson/console/AnnotatedLargeText.java
            trunk/hudson/main/core/src/main/java/hudson/console/ConsoleNote.java
            trunk/hudson/main/core/src/main/java/hudson/console/PlainTextConsoleOutputStream.java
            trunk/hudson/main/core/src/main/java/hudson/util/UnbufferedBase64InputStream.java
            trunk/hudson/main/test/src/test/java/hudson/console/ConsoleAnnotatorTest.java
            trunk/www/changelog.html
            http://jenkins-ci.org/commit/32514
            Log:
            [FIXED JENKINS-6034] plain text console output needs to filter out inline console annotations

            Show
            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : kohsuke Path: trunk/hudson/main/core/src/main/java/hudson/console/AnnotatedLargeText.java trunk/hudson/main/core/src/main/java/hudson/console/ConsoleNote.java trunk/hudson/main/core/src/main/java/hudson/console/PlainTextConsoleOutputStream.java trunk/hudson/main/core/src/main/java/hudson/util/UnbufferedBase64InputStream.java trunk/hudson/main/test/src/test/java/hudson/console/ConsoleAnnotatorTest.java trunk/www/changelog.html http://jenkins-ci.org/commit/32514 Log: [FIXED JENKINS-6034] plain text console output needs to filter out inline console annotations

              People

              • Assignee:
                kohsuke Kohsuke Kawaguchi
                Reporter:
                jglick Jesse Glick
              • Votes:
                2 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: