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

JaCoCo 2.0.0 plugin shows html instead of coverage report chart

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Critical
    • Resolution: Fixed
    • jacoco-plugin
    • None
    • Jenkins 1.625.2
      JaCoCo plugin 2.0.0
      Git plugin 2.4.0
      Git client plugin 1.19.0

    Description

      The jacoco plugin shows raw HTML instead of showing the coverage chart in certain views.

      Steps to duplicate the problem:

      • Download the reference docker image including the git-client-coverage job and plugin configurations
            git clone git://github.com/MarkEWaite/docker.git
            cd docker
            git checkout -b master-jacoco-2.0.0-bug -t origin/master-jacoco-2.0.0-bug
            sudo docker build -t master-jacoco-2.0.0-bug .
            sudo docker run -it -p 9090:8080 -p 60000:50000 master-jacoco-2.0.0-bug
            
      • Build git-client-coverage job once
      • Select the left axis of the "Code Coverage Trend" graph

      The page which appears will show raw HTML instead of showing the expected chart or table. Refer to the attached picture to see the results.

      Attachments

        Issue Links

          Activity

            Code changed in jenkins
            User: Calin Iorgulescu
            Path:
            src/main/resources/hudson/plugins/jacoco/tags/breakdownMethodsTable.jelly
            src/main/resources/hudson/plugins/jacoco/tags/breakdownTable.jelly
            src/main/resources/hudson/plugins/jacoco/tags/summaryMethod.jelly
            src/main/resources/hudson/plugins/jacoco/tags/summaryTable.jelly
            http://jenkins-ci.org/commit/jacoco-plugin/8651004c738ea37f43932cea0de244d20b16d52f
            Log:
            [FIXED JENKINS-31751]: Correctly escape HTML in jelly files.

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Calin Iorgulescu Path: src/main/resources/hudson/plugins/jacoco/tags/breakdownMethodsTable.jelly src/main/resources/hudson/plugins/jacoco/tags/breakdownTable.jelly src/main/resources/hudson/plugins/jacoco/tags/summaryMethod.jelly src/main/resources/hudson/plugins/jacoco/tags/summaryTable.jelly http://jenkins-ci.org/commit/jacoco-plugin/8651004c738ea37f43932cea0de244d20b16d52f Log: [FIXED JENKINS-31751] : Correctly escape HTML in jelly files.

            Code changed in jenkins
            User: Dominik Stadler
            Path:
            src/main/resources/hudson/plugins/jacoco/tags/breakdownMethodsTable.jelly
            src/main/resources/hudson/plugins/jacoco/tags/breakdownTable.jelly
            src/main/resources/hudson/plugins/jacoco/tags/summaryMethod.jelly
            src/main/resources/hudson/plugins/jacoco/tags/summaryTable.jelly
            http://jenkins-ci.org/commit/jacoco-plugin/c5edf40d484ec11a2ccbc20f73b2c482dbf8c261
            Log:
            Merge pull request #62 from calin-iorgulescu/master

            JENKINS-31751: Fix HTML rendering by correctly escaping it in jelly files.

            Compare: https://github.com/jenkinsci/jacoco-plugin/compare/5c79c7896ba9...c5edf40d484e

            scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Dominik Stadler Path: src/main/resources/hudson/plugins/jacoco/tags/breakdownMethodsTable.jelly src/main/resources/hudson/plugins/jacoco/tags/breakdownTable.jelly src/main/resources/hudson/plugins/jacoco/tags/summaryMethod.jelly src/main/resources/hudson/plugins/jacoco/tags/summaryTable.jelly http://jenkins-ci.org/commit/jacoco-plugin/c5edf40d484ec11a2ccbc20f73b2c482dbf8c261 Log: Merge pull request #62 from calin-iorgulescu/master JENKINS-31751 : Fix HTML rendering by correctly escaping it in jelly files. Compare: https://github.com/jenkinsci/jacoco-plugin/compare/5c79c7896ba9...c5edf40d484e
            dantran dan tran added a comment -

            It works!!! tested with the latest snapshot. Release is coming soon?

            dantran dan tran added a comment - It works!!! tested with the latest snapshot. Release is coming soon?
            martinotu martino tu added a comment -

            Any update on this? Thanks!

            martinotu martino tu added a comment - Any update on this? Thanks!
            centic centic added a comment -

            Version 2.0.1 is published and should be available via updates soon.

            centic centic added a comment - Version 2.0.1 is published and should be available via updates soon.
            dhubau Dieter Hubau added a comment -

            Confirmed that version 2.0.1 fixed the problem, thanks!

            dhubau Dieter Hubau added a comment - Confirmed that version 2.0.1 fixed the problem, thanks!

            People

              ognjenb Ognjen Bubalo
              markewaite Mark Waite
              Votes:
              15 Vote for this issue
              Watchers:
              21 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: