Fixed jenkinsci#144

khmarbaise · khmarbaise · commit d46442ea4252 · 2016-04-11T08:36:22.000+02:00
o Problems with spaces in paths and job names
 o Reported by Shiv Kaushal
diff --git a/ReleaseNotes.md b/ReleaseNotes.md
@@ -21,6 +21,8 @@ void enableJob(String jobName, boolean crumbFlag);
   which uses the XStream classes which does not make sense
   from my point of view.
 
+  [Fixed #144 Problems with spaces in paths and job names][issue-144]
+
 
 ## Release 0.3.3
 
@@ -227,5 +229,6 @@ TestReport testReport = mavenJob.getLastSuccessfulBuild().getTestReport();
 [issue-116]: https://github.com/RisingOak/jenkins-client/issues/116
 [issue-108]: https://github.com/RisingOak/jenkins-client/issues/108
 [issue-130]: https://github.com/RisingOak/jenkins-client/issues/130
+[issue-144]: https://github.com/RisingOak/jenkins-client/issues/144
 [pull-123]: https://github.com/RisingOak/jenkins-client/pull/123
 
diff --git a/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java b/src/main/java/com/offbytwo/jenkins/client/JenkinsHttpClient.java
@@ -458,7 +458,7 @@ private URI api(String path) {
             String[] components = path.split("\\?", 2);
             path = urlJoin(components[0], "api/json") + "?" + components[1];
         }
-        return uri.resolve("/").resolve(path);
+        return uri.resolve("/").resolve(path.replace(" ","%20"));
     }
 
     private URI noapi(String path) {

Original file line number	Diff line number	Diff line change
`@@ -458,7 +458,7 @@ private URI api(String path) {`
`458`	`458`	`String[] components = path.split("\\?", 2);`
`459`	`459`	`path = urlJoin(components[0], "api/json") + "?" + components[1];`
`460`	`460`	`}`
`461`		`- return uri.resolve("/").resolve(path);`
	`461`	`+ return uri.resolve("/").resolve(path.replace(" ","%20"));`
`462`	`462`	`}`
`463`	`463`
`464`	`464`	`private URI noapi(String path) {`