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

Should trim space from directory entry on HTML publisher

    • 1.15

      I was getting this error after installing the HTML Publisher:

      [htmlpublisher] Archiving HTML reports...
      [htmlpublisher] Archiving at PROJECT level
      /var/lib/jenkins/workspace/simplesimple/reports/html/ to /var/lib/jenkins/jobs/simplesimple/htmlreports/HTML_Report
      ERROR: Specified HTML directory '/var/lib/jenkins/workspace/simplesimple/reports/html/ ' does not exist.
      Build step 'Publish HTML reports' changed build result to FAILURE

      The problem was, I had an additional space character at the end of my directory entry, which wasn't obvious from the interface.

      It would be great to update this to trim leading/trailing whitespace from directory names.

          [JENKINS-47034] Should trim space from directory entry on HTML publisher

          I have a PR prepared ready for merge (https://github.com/jenkinsci/htmlpublisher-plugin/pull/30) that will trim when the configuration is saved to help prevent this issue. I've got a few more issues I want to work through before doing a new release but hopefully it won't be too far away from getting released.

          Richard Bywater added a comment - I have a PR prepared ready for merge ( https://github.com/jenkinsci/htmlpublisher-plugin/pull/30 ) that will trim when the configuration is saved to help prevent this issue. I've got a few more issues I want to work through before doing a new release but hopefully it won't be too far away from getting released.

          Code changed in jenkins
          User: Richard Bywater
          Path:
          src/main/java/htmlpublisher/HtmlPublisherTarget.java
          src/test/groovy/htmlpublisher/HtmlPublisherIntegrationTest.groovy
          src/test/groovy/htmlpublisher/HtmlPublisherTest.groovy
          src/test/java/htmlpublisher/HtmlPublisherTest.java
          http://jenkins-ci.org/commit/htmlpublisher-plugin/807c7585960e46f0e7e89dfc3956525e0e32cbfd
          Log:
          Fix JENKINS-47034

          Report Name, directory, files, and titles are now all trimmed. Note that
          trimming is only done when configuration is saved.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Richard Bywater Path: src/main/java/htmlpublisher/HtmlPublisherTarget.java src/test/groovy/htmlpublisher/HtmlPublisherIntegrationTest.groovy src/test/groovy/htmlpublisher/HtmlPublisherTest.groovy src/test/java/htmlpublisher/HtmlPublisherTest.java http://jenkins-ci.org/commit/htmlpublisher-plugin/807c7585960e46f0e7e89dfc3956525e0e32cbfd Log: Fix JENKINS-47034 Report Name, directory, files, and titles are now all trimmed. Note that trimming is only done when configuration is saved.

          Code changed in jenkins
          User: Richard Bywater
          Path:
          src/main/java/htmlpublisher/HtmlPublisherTarget.java
          src/test/groovy/htmlpublisher/HtmlPublisherIntegrationTest.groovy
          src/test/groovy/htmlpublisher/HtmlPublisherTest.groovy
          src/test/java/htmlpublisher/HtmlPublisherTest.java
          http://jenkins-ci.org/commit/htmlpublisher-plugin/f187d56a4cd3fdcba69d0abc9deaac5be072d2d5
          Log:
          Merge pull request #30 from rbywater/bugfix/JENKINS-47034

          Fix JENKINS-47034

          Compare: https://github.com/jenkinsci/htmlpublisher-plugin/compare/b97dafba306e...f187d56a4cd3

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Richard Bywater Path: src/main/java/htmlpublisher/HtmlPublisherTarget.java src/test/groovy/htmlpublisher/HtmlPublisherIntegrationTest.groovy src/test/groovy/htmlpublisher/HtmlPublisherTest.groovy src/test/java/htmlpublisher/HtmlPublisherTest.java http://jenkins-ci.org/commit/htmlpublisher-plugin/f187d56a4cd3fdcba69d0abc9deaac5be072d2d5 Log: Merge pull request #30 from rbywater/bugfix/ JENKINS-47034 Fix JENKINS-47034 Compare: https://github.com/jenkinsci/htmlpublisher-plugin/compare/b97dafba306e...f187d56a4cd3

          This fix will be going out with the next version which will be in the new year so as not to break installations whilst support is potentially unavailable over the Christmas period.

          Richard Bywater added a comment - This fix will be going out with the next version which will be in the new year so as not to break installations whilst support is potentially unavailable over the Christmas period.

          This was actually fixed quit a while ago but never closed the issue!

          Richard Bywater added a comment - This was actually fixed quit a while ago but never closed the issue!

            r2b2_nz Richard Bywater
            kewob Karl Brown
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: