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

test harness depends on buggy XML parsers, causes ItemGroupMixIn.createProjectFromXML() to fail if path contains a space

      Seen this when saving through job-dsl plugin with a folder with spaces in it.

      java.io.IOException: Failed to persist config.xml
      

      This looks like a bug in the interface in StreamResult in java's transformer library:

      http://stackoverflow.com/questions/20434510/transformer-library-that-comes-with-java-converts-spaces-in-file-paths-to-20

          [JENKINS-35638] test harness depends on buggy XML parsers, causes ItemGroupMixIn.createProjectFromXML() to fail if path contains a space

          Dan Alvizu added a comment -

          Dan Alvizu added a comment - PR: https://github.com/jenkinsci/jenkins/pull/2404

          Dan Alvizu added a comment -

          Originally assigned to 'INFRA' project and R Tyler Croy - sorry about that. Re auto-assigning.

          Dan Alvizu added a comment - Originally assigned to 'INFRA' project and R Tyler Croy - sorry about that. Re auto-assigning.

          Iyad Al added a comment - - edited

          How long will it roughly take to have a release for this bug fix?

          Iyad Al added a comment - - edited How long will it roughly take to have a release for this bug fix?

          James Nord added a comment -

          This is unreproducable in Jenkins.

           

          The only reproducable test case for this has been in a unit test for running a job dsl script using JenkinsRule and the jenkins-test-harness pulls in buggy xml parsers.  Excluding these shows that the problem is resolved.

           

          Moving to jenkins-test-harness component.

          James Nord added a comment - This is unreproducable in Jenkins.   The only reproducable test case for this has been in a unit test for running a job dsl script using JenkinsRule and the jenkins-test-harness pulls in buggy xml parsers.  Excluding these shows that the problem is resolved.   Moving to jenkins-test-harness component.

          James Nord added a comment -

          see https://github.com/jgangemi/jenkins-35638-issue/pull/1 for a test case and the demonstrated fix (excluding the parsers)

          James Nord added a comment - see https://github.com/jgangemi/jenkins-35638-issue/pull/1  for a test case and the demonstrated fix (excluding the parsers)

          Jesse Glick added a comment -

          From last comments, sounds like this is not an actual issue.

          Jesse Glick added a comment - From last comments, sounds like this is not an actual issue.

            Unassigned Unassigned
            dalvizu Dan Alvizu
            Votes:
            3 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: