Use helidon-common-config where possible/applicable (helidon-io#6448)

Author: arjav-desai

common/config/src/main/java/io/helidon/common/config/Config.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2018, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2018, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -225,6 +225,18 @@ default Config get(Key key) {
      */
     <T> ConfigValue<List<T>> asList(Class<T> type) throws ConfigException;
 
+    /**
+     * Returns a list of child {@code Config} nodes if the node is {@code Type#OBJECT}.
+     * Returns a list of element nodes if the node is {@code Type#LIST}.
+     * Throws {@code MissingValueException} if the node is {@code Type#MISSING}.
+     * Otherwise, if node is {@code Type#VALUE}, it throws {@code ConfigMappingException}.
+     *
+     * @return a list of {@code Type#OBJECT} members or a list of {@code Type#LIST} members
+     * @param <C> the common config derived type
+     * @throws io.helidon.common.config.ConfigException in case the node is {@code Type#VALUE}
+     */
+    <C extends Config> ConfigValue<List<C>> asNodeList() throws ConfigException;
+
     /**
      * Transform all leaf nodes (values) into Map instance.
      *

config/config/src/main/java/io/helidon/config/Config.java

@@ -827,15 +827,8 @@ default ConfigValue<Config> asNode() {
                                    Config::asNode);
     }
 
-    /**
-     * Returns a list of child {@code Config} nodes if the node is {@link Type#OBJECT}.
-     * Returns a list of element nodes if the node is {@link Type#LIST}.
-     * Throws {@link MissingValueException} if the node is {@link Type#MISSING}.
-     * Otherwise, if node is {@link Type#VALUE}, it throws {@link ConfigMappingException}.
-     *
-     * @return a list of {@link Type#OBJECT} members or a list of {@link Type#LIST} members
-     * @throws ConfigMappingException in case the node is {@link Type#VALUE}
-     */
+    @Override
+    @SuppressWarnings("unchecked")
     ConfigValue<List<Config>> asNodeList() throws ConfigMappingException;
 
     /**

cors/pom.xml

@@ -31,8 +31,8 @@
 
     <dependencies>
         <dependency>
-            <groupId>io.helidon.config</groupId>
-            <artifactId>helidon-config</artifactId>
+            <groupId>io.helidon.common</groupId>
+            <artifactId>helidon-common-config</artifactId>
         </dependency>
         <dependency>
             <groupId>io.helidon.common</groupId>

cors/src/main/java/io/helidon/cors/Aggregator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,11 +22,11 @@
 import java.util.function.Supplier;
 import java.util.logging.Logger;
 
+import io.helidon.common.config.Config;
+import io.helidon.common.config.ConfigValue;
 import io.helidon.common.http.PathMatcher;
 import io.helidon.common.http.PathMatchers;
 import io.helidon.common.uri.UriPath;
-import io.helidon.config.Config;
-import io.helidon.config.ConfigValue;
 import io.helidon.cors.LogHelper.MatcherChecks;
 
 /**
@@ -118,7 +118,7 @@ public Aggregator build() {
 
         Builder config(Config config) {
             if (config.exists()) {
-                ConfigValue<CrossOriginConfig.Builder> configValue = config.as(CrossOriginConfig::builder);
+                ConfigValue<CrossOriginConfig.Builder> configValue = config.map(CrossOriginConfig::builder);
                 if (configValue.isPresent()) {
                     CrossOriginConfig crossOriginConfig = configValue.get().build();
                     addPathlessCrossOrigin(crossOriginConfig);
@@ -136,7 +136,7 @@ Builder config(Config config) {
         Builder mappedConfig(Config config) {
 
             if (config.exists()) {
-                ConfigValue<MappedCrossOriginConfig.Builder> mappedConfigValue = config.as(MappedCrossOriginConfig::builder);
+                ConfigValue<MappedCrossOriginConfig.Builder> mappedConfigValue = config.map(MappedCrossOriginConfig::builder);
                 if (mappedConfigValue.isPresent()) {
                     MappedCrossOriginConfig mapped = mappedConfigValue.get().build();
                     /*

cors/src/main/java/io/helidon/cors/CorsSupportBase.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -21,7 +21,7 @@
 import java.util.logging.Level;
 import java.util.logging.Logger;
 
-import io.helidon.config.Config;
+import io.helidon.common.config.Config;
 
 /**
  * A Helidon service and handler implementation that implements CORS, for both the application and for built-in Helidon

cors/src/main/java/io/helidon/cors/CorsSupportHelper.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -28,9 +28,9 @@
 import java.util.function.Supplier;
 import java.util.logging.Logger;
 
+import io.helidon.common.config.Config;
 import io.helidon.common.http.Http;
 import io.helidon.common.http.Http.Header;
-import io.helidon.config.Config;
 import io.helidon.cors.LogHelper.Headers;
 
 import static io.helidon.cors.LogHelper.DECISION_LEVEL;

cors/src/main/java/io/helidon/cors/CrossOriginConfig.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -18,7 +18,7 @@
 
 import java.util.Arrays;
 
-import io.helidon.config.Config;
+import io.helidon.common.config.Config;
 import io.helidon.config.metadata.Configured;
 import io.helidon.config.metadata.ConfiguredOption;
 

cors/src/main/java/io/helidon/cors/Loader.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,8 +15,8 @@
  */
 package io.helidon.cors;
 
-import io.helidon.config.Config;
-import io.helidon.config.ConfigValue;
+import io.helidon.common.config.Config;
+import io.helidon.common.config.ConfigValue;
 
 import static io.helidon.cors.Aggregator.PATHLESS_KEY;
 import static io.helidon.cors.CrossOriginConfig.CORS_PATHS_CONFIG_KEY;
@@ -78,7 +78,7 @@ static MappedCrossOriginConfig.Builder applyConfig(MappedCrossOriginConfig.Build
                 if (!item.exists()) {
                     break;
                 }
-                ConfigValue<CrossOriginConfig.Builder> basicConfigValue = item.as(CrossOriginConfig::builder);
+                ConfigValue<CrossOriginConfig.Builder> basicConfigValue = item.map(CrossOriginConfig::builder);
                 if (!basicConfigValue.isPresent()) {
                     continue;
                 }

cors/src/main/java/io/helidon/cors/MappedCrossOriginConfig.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2020, 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2020, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -22,7 +22,7 @@
 import java.util.Optional;
 import java.util.function.BiConsumer;
 
-import io.helidon.config.Config;
+import io.helidon.common.config.Config;
 
 /**
  * Cross-origin {@link CrossOriginConfig} instances linked to paths, plus an {@code enabled} setting. Most developers will not

cors/src/main/java/module-info.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2022 Oracle and/or its affiliates.
+ * Copyright (c) 2022, 2023 Oracle and/or its affiliates.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -20,7 +20,7 @@
 module io.helidon.cors {
     requires java.logging;
     requires io.helidon.common.http;
-    requires io.helidon.config;
+    requires io.helidon.common.config;
 
     requires static io.helidon.config.metadata;