From 8b5247bc625506ec86d6e711581ea5e944bc68b9 Mon Sep 17 00:00:00 2001
From: chunweilim <cwlim1995@hotmail.com>
Date: Fri, 14 Jul 2023 00:59:45 +1000
Subject: [PATCH] Add setter to auths if it's null to avoid NPE when getting
 auths

---
 .../com/github/dockerjava/core/DockerConfigFile.java     | 8 +++++++-
 .../com/github/dockerjava/core/DockerConfigFileTest.java | 6 ++++++
 .../testAuthConfigFile/validJsonAuthsNull/config.json    | 9 +++++++++
 3 files changed, 22 insertions(+), 1 deletion(-)
 create mode 100644 docker-java/src/test/resources/testAuthConfigFile/validJsonAuthsNull/config.json

diff --git a/docker-java-core/src/main/java/com/github/dockerjava/core/DockerConfigFile.java b/docker-java-core/src/main/java/com/github/dockerjava/core/DockerConfigFile.java
index 825796a74..39ef15271 100644
--- a/docker-java-core/src/main/java/com/github/dockerjava/core/DockerConfigFile.java
+++ b/docker-java-core/src/main/java/com/github/dockerjava/core/DockerConfigFile.java
@@ -1,6 +1,7 @@
 package com.github.dockerjava.core;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+import com.fasterxml.jackson.annotation.JsonSetter;
 import com.fasterxml.jackson.core.type.TypeReference;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.github.dockerjava.api.model.AuthConfig;
@@ -28,7 +29,7 @@ public class DockerConfigFile {
     };
 
     @JsonProperty
-    private final Map<String, AuthConfig> auths;
+    private Map<String, AuthConfig> auths;
 
     @JsonProperty
     private String currentContext;
@@ -46,6 +47,11 @@ public Map<String, AuthConfig> getAuths() {
         return auths;
     }
 
+    @JsonSetter
+    public void setAuths(Map<String, AuthConfig> authConfigMap) {
+        auths = (authConfigMap == null || authConfigMap.size() == 0) ? new HashMap<>() : authConfigMap;
+    }
+
     void addAuthConfig(AuthConfig config) {
         auths.put(config.getRegistryAddress(), config);
     }
diff --git a/docker-java/src/test/java/com/github/dockerjava/core/DockerConfigFileTest.java b/docker-java/src/test/java/com/github/dockerjava/core/DockerConfigFileTest.java
index ce1a59cc0..76211fc55 100644
--- a/docker-java/src/test/java/com/github/dockerjava/core/DockerConfigFileTest.java
+++ b/docker-java/src/test/java/com/github/dockerjava/core/DockerConfigFileTest.java
@@ -164,6 +164,12 @@ public void nonExistent() throws IOException {
         assertThat(runTest("idontexist"), is(expected));
     }
 
+    @Test
+    public void validJsonAuthsNull() throws IOException {
+        DockerConfigFile expected = new DockerConfigFile();
+        assertThat(runTest("validJsonAuthsNull"), is(expected));
+    }
+
     private DockerConfigFile runTest(String testFileName) throws IOException {
         return DockerConfigFile.loadConfig(JSONTestHelper.getMapper(), new File(FILESROOT, testFileName).getAbsolutePath());
     }
diff --git a/docker-java/src/test/resources/testAuthConfigFile/validJsonAuthsNull/config.json b/docker-java/src/test/resources/testAuthConfigFile/validJsonAuthsNull/config.json
new file mode 100644
index 000000000..d104c357c
--- /dev/null
+++ b/docker-java/src/test/resources/testAuthConfigFile/validJsonAuthsNull/config.json
@@ -0,0 +1,9 @@
+{
+    "auths": null,
+    "credsStore": "desktop",
+    "plugins": {
+        "-x-cli-hints": {
+            "enabled": "true"
+        }
+    }
+}