diff --git a/src/main/java/com/github/dockerjava/core/DockerClientConfig.java b/src/main/java/com/github/dockerjava/core/DockerClientConfig.java index 038c43cae..34ce13133 100644 --- a/src/main/java/com/github/dockerjava/core/DockerClientConfig.java +++ b/src/main/java/com/github/dockerjava/core/DockerClientConfig.java @@ -13,7 +13,6 @@ import java.util.Properties; import java.util.Set; -import org.apache.commons.lang.BooleanUtils; import org.apache.commons.lang.StringUtils; import org.apache.commons.lang.builder.EqualsBuilder; import org.apache.commons.lang.builder.HashCodeBuilder; @@ -418,8 +417,8 @@ public final DockerClientConfigBuilder withDockerConfig(String dockerConfig) { } public final DockerClientConfigBuilder withDockerTlsVerify(String dockerTlsVerify) { - this.dockerTlsVerify = BooleanUtils.toBoolean(dockerTlsVerify.trim()) - || BooleanUtils.toBoolean(dockerTlsVerify.trim(), "1", "0"); + String trimmed = dockerTlsVerify.trim(); + this.dockerTlsVerify = "true".equals(trimmed) || "1".equals(trimmed); return this; } diff --git a/src/test/java/com/github/dockerjava/core/DockerClientConfigTest.java b/src/test/java/com/github/dockerjava/core/DockerClientConfigTest.java index e87254b88..af42df640 100644 --- a/src/test/java/com/github/dockerjava/core/DockerClientConfigTest.java +++ b/src/test/java/com/github/dockerjava/core/DockerClientConfigTest.java @@ -3,7 +3,10 @@ import static org.hamcrest.MatcherAssert.assertThat; import static org.hamcrest.Matchers.equalTo; import static org.testng.Assert.assertEquals; +import static org.testng.Assert.assertFalse; +import static org.testng.Assert.assertTrue; +import java.lang.reflect.Field; import java.net.URI; import java.util.Collections; import java.util.HashMap; @@ -176,4 +179,23 @@ public void testUnixHostScheme() throws Exception { new DockerClientConfig(URI.create("unix://foo"), "dockerConfig", "apiVersion", "registryUrl", "registryUsername", "registryPassword", "registryEmail", null, false); } + + @Test + public void withDockerTlsVerify() throws Exception { + DockerClientConfig.DockerClientConfigBuilder builder = new DockerClientConfig.DockerClientConfigBuilder(); + Field field = builder.getClass().getDeclaredField("dockerTlsVerify"); + field.setAccessible(true); + + builder.withDockerTlsVerify(""); + assertFalse(field.getBoolean(builder)); + builder.withDockerTlsVerify("false"); + assertFalse(field.getBoolean(builder)); + builder.withDockerTlsVerify("true"); + assertTrue(field.getBoolean(builder)); + builder.withDockerTlsVerify("0"); + assertFalse(field.getBoolean(builder)); + builder.withDockerTlsVerify("1"); + assertTrue(field.getBoolean(builder)); + } + }