[Dubbo-DynamicConfig] compatible configurators url with type JsonArray in ConfigCenter

dubbo-all/pom.xml

@@ -570,6 +570,10 @@
             <groupId>org.yaml</groupId>
             <artifactId>snakeyaml</artifactId>
         </dependency>
+        <dependency>
+            <groupId>com.alibaba</groupId>
+            <artifactId>fastjson</artifactId>
+        </dependency>
 
         <!-- Temporarily add this part to exclude transitive dependency -->
         <dependency>

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/ConfigParser.java

@@ -16,6 +16,8 @@
  */
 package org.apache.dubbo.rpc.cluster.configurator.parser;
 
+import com.alibaba.fastjson.JSON;
+import com.alibaba.fastjson.JSONValidator;
 import org.apache.dubbo.common.URL;
 import org.apache.dubbo.common.utils.CollectionUtils;
 import org.apache.dubbo.common.utils.StringUtils;
@@ -41,6 +43,11 @@
 public class ConfigParser {
 
     public static List<URL> parseConfigurators(String rawConfig) {
+        // compatible url JsonArray, such as [ "override://xxx", "override://xxx" ]
+        if (isJsonArray(rawConfig)) {
+            return parseJsonArray(rawConfig);
+        }
+
         List<URL> urls = new ArrayList<>();
         ConfiguratorConfig configuratorConfig = parseObject(rawConfig);
 
@@ -56,6 +63,15 @@ public static List<URL> parseConfigurators(String rawConfig) {
         return urls;
     }
 
+    private static List<URL> parseJsonArray(String rawConfig) {
+        List<URL> urls = new ArrayList<>();
+        List<String> list = JSON.parseArray(rawConfig, String.class);
+        if (!CollectionUtils.isEmpty(list)) {
+            list.forEach(u -> urls.add(URL.valueOf(u)));
+        }
+        return urls;
+    }
+
     private static <T> T parseObject(String rawConfig) {
         Constructor constructor = new Constructor(ConfiguratorConfig.class);
         TypeDescription itemDescription = new TypeDescription(ConfiguratorConfig.class);
@@ -200,4 +216,14 @@ private static List<String> parseAddresses(ConfigItem item) {
         }
         return addresses;
     }
+
+    private static boolean isJsonArray(String rawConfig) {
+        try {
+            JSONValidator validator = JSONValidator.from(rawConfig);
+            return validator.validate() && validator.getType() == JSONValidator.Type.Array;
+        } catch (Exception e) {
+            // ignore exception and return false
+        }
+        return false;
+    }
 }

dubbo-cluster/src/test/java/org/apache/dubbo/rpc/cluster/configurator/parser/ConfigParserTest.java

@@ -174,4 +174,20 @@ public void parseConsumerSpecificProvidersTest() throws IOException {
         }
     }
 
+    @Test
+    public void parseURLJsonArrayCompatible() {
+
+        String configData = "[\"override://0.0.0.0/com.xx.Service?category=configurators&timeout=6666&disabled=true&dynamic=false&enabled=true&group=dubbo&priority=1&version=1.0\" ]";
+
+        List<URL> urls = ConfigParser.parseConfigurators(configData);
+
+        Assertions.assertNotNull(urls);
+        Assertions.assertEquals(1, urls.size());
+        URL url = urls.get(0);
+
+        Assertions.assertEquals("0.0.0.0", url.getAddress());
+        Assertions.assertEquals("com.xx.Service", url.getServiceInterface());
+        Assertions.assertEquals(6666, url.getParameter(TIMEOUT_KEY, 0));
+    }
+
 }