Only override AuthConfig iff username/password/email/server address are all set

tedoc2000 · tedoc2000 · commit 6fa4cc0225a5 · 2015-02-05T07:59:20.000-08:00
Unless this is done, it is possible to create an invalid AuthConfig with null
values for password and other fields.
diff --git a/src/main/java/com/github/dockerjava/core/DockerClientConfig.java b/src/main/java/com/github/dockerjava/core/DockerClientConfig.java
@@ -253,7 +253,8 @@ public String getDockerCfgPath() {
     
     private AuthConfig getAuthConfig() {
     	AuthConfig authConfig = null;
-    	if(getUsername() != null) {
+    	if (getUsername() != null && getPassword() != null && getEmail() != null
+    			&& getServerAddress() != null) {
     		authConfig = new AuthConfig();
     		authConfig.setUsername(getUsername());
     		authConfig.setPassword(getPassword());
