Encode spaces as %20 rather than + in URL params

orzeh · KostyaSha · commit a8fa3837f7d6 · 2018-07-23T04:34:05.000+03:00
diff --git a/src/main/java/com/github/dockerjava/jaxrs/BuildImageCmdExec.java b/src/main/java/com/github/dockerjava/jaxrs/BuildImageCmdExec.java
@@ -141,7 +141,7 @@ private WebTarget writeMap(WebTarget webTarget, String name, Map<String, String>
         if (value != null && !value.isEmpty()) {
             try {
                 return webTarget.queryParam(name,
-                        URLEncoder.encode(MAPPER.writeValueAsString(value), "UTF-8"));
+                        URLEncoder.encode(MAPPER.writeValueAsString(value), "UTF-8").replaceAll("\\+", "%20"));
             } catch (IOException e) {
                 throw new RuntimeException(e);
             }
diff --git a/src/test/java/com/github/dockerjava/cmd/BuildImageCmdIT.java b/src/test/java/com/github/dockerjava/cmd/BuildImageCmdIT.java
@@ -240,15 +240,15 @@ public void fromPrivateRegistry() throws Exception {
     public void buildArgs() throws Exception {
         File baseDir = fileFromBuildTestResource("buildArgs");
 
-        String imageId = dockerRule.getClient().buildImageCmd(baseDir).withNoCache(true).withBuildArg("testArg", "abc")
+        String imageId = dockerRule.getClient().buildImageCmd(baseDir).withNoCache(true).withBuildArg("testArg", "abc !@#$%^&*()_+")
                 .exec(new BuildImageResultCallback())
                 .awaitImageId();
 
         InspectImageResponse inspectImageResponse = dockerRule.getClient().inspectImageCmd(imageId).exec();
         assertThat(inspectImageResponse, not(nullValue()));
         LOG.info("Image Inspect: {}", inspectImageResponse.toString());
 
-        assertThat(inspectImageResponse.getConfig().getLabels().get("test"), equalTo("abc"));
+        assertThat(inspectImageResponse.getConfig().getLabels().get("test"), equalTo("abc !@#$%^&*()_+"));
     }
 
     @Test

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ private WebTarget writeMap(WebTarget webTarget, String name, Map<String, String>`
`141`	`141`	`if (value != null && !value.isEmpty()) {`
`142`	`142`	`try {`
`143`	`143`	`return webTarget.queryParam(name,`
`144`		`- URLEncoder.encode(MAPPER.writeValueAsString(value), "UTF-8"));`
	`144`	`+ URLEncoder.encode(MAPPER.writeValueAsString(value), "UTF-8").replaceAll("\\+", "%20"));`
`145`	`145`	`} catch (IOException e) {`
`146`	`146`	`throw new RuntimeException(e);`
`147`	`147`	`}`