Avoid NPE when call AbstractDynamicConfiguration#execute (apache#6576)

fix apache#6575
bouxin · Aug 14, 2020 · 56353ae · 56353ae
1 parent 683db2b
commit 56353ae
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...c/main/java/org/apache/dubbo/common/config/configcenter/AbstractDynamicConfiguration.java b/...c/main/java/org/apache/dubbo/common/config/configcenter/AbstractDynamicConfiguration.java
@@ -130,7 +130,7 @@ public final void close() throws Exception {
 
     @Override
     public boolean removeConfig(String key, String group) {
-        return execute(() -> doRemoveConfig(key, group), -1L);
+        return Boolean.TRUE.equals(execute(() -> doRemoveConfig(key, group), -1L));
     }
 
     /**

diff --git a/...c/main/java/org/apache/dubbo/common/config/configcenter/TreePathDynamicConfiguration.java b/...c/main/java/org/apache/dubbo/common/config/configcenter/TreePathDynamicConfiguration.java
@@ -87,7 +87,7 @@ protected final String doGetConfig(String key, String group) throws Exception {
     @Override
     public final boolean publishConfig(String key, String group, String content) {
         String pathKey = buildPathKey(group, key);
-        return execute(() -> doPublishConfig(pathKey, content), getDefaultTimeout());
+        return Boolean.TRUE.equals(execute(() -> doPublishConfig(pathKey, content), getDefaultTimeout()));
     }
 
     @Override