Add debug logging

Author: adrienlauer

CHANGELOG.md

@@ -1,3 +1,7 @@
+# Version 3.1.2 (2018-10-18)
+
+* [chg] Add debug logging
+
 # Version 3.1.1 (2018-09-03)
 
 * [fix] Always make ServiceLoader use the most complete classloader it can find.

pom.xml

@@ -19,7 +19,7 @@
 
     <groupId>org.seedstack.coffig</groupId>
     <artifactId>coffig</artifactId>
-    <version>3.1.1-SNAPSHOT</version>
+    <version>3.1.2-SNAPSHOT</version>
 
     <properties>
         <shed.version>1.1.3</shed.version>

src/main/java/org/seedstack/coffig/CoffigBuilder.java

@@ -112,11 +112,11 @@ public Coffig build() {
 
         return new Coffig(
                 wrap(new EvaluatingMapper(
-                        new CompositeMapper(mappers.toArray(new ConfigurationMapper[mappers.size()])),
-                        new CompositeEvaluator(evaluators.toArray(new ConfigurationEvaluator[evaluators.size()]))
+                        new CompositeMapper(mappers.toArray(new ConfigurationMapper[0])),
+                        new CompositeEvaluator(evaluators.toArray(new ConfigurationEvaluator[0]))
                 )),
-                new CompositeProvider(providers.toArray(new ConfigurationProvider[providers.size()])),
-                new CompositeProcessor(processors.toArray(new ConfigurationProcessor[processors.size()]))
+                new CompositeProvider(providers.toArray(new ConfigurationProvider[0])),
+                new CompositeProcessor(processors.toArray(new ConfigurationProcessor[0]))
         );
     }
 

src/main/java/org/seedstack/coffig/provider/EnvironmentProvider.java

@@ -11,10 +11,15 @@
 import org.seedstack.coffig.node.MapNode;
 import org.seedstack.coffig.node.NamedNode;
 import org.seedstack.coffig.spi.ConfigurationProvider;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class EnvironmentProvider implements ConfigurationProvider {
+    private static final Logger LOGGER = LoggerFactory.getLogger(EnvironmentProvider.class);
+
     @Override
     public MapNode provide() {
+        LOGGER.debug("Reading configuration from environment variables");
         return new MapNode(System.getenv().entrySet().stream()
                 .map(e -> new NamedNode(e.getKey(), e.getValue()))
                 .toArray(NamedNode[]::new));

src/main/java/org/seedstack/coffig/provider/JacksonProvider.java

@@ -91,12 +91,14 @@ public synchronized JacksonProvider addSource(URL url) {
     }
 
     @Override
-    public void fileChanged(Path changed) {
+    public void fileChanged(Path path) {
+        LOGGER.debug("Configuration file has changed: " + path);
         dirty.set(true);
     }
 
     private MapNode buildTreeFromSource(URL url) {
         try {
+            LOGGER.debug("Reading configuration from " + url.toExternalForm());
             return buildTreeFromFields(jacksonMapper.readTree(url));
         } catch (IOException e) {
             throw ConfigurationException.wrap(e, ConfigurationErrorCode.FAILED_TO_READ_CONFIGURATION)
@@ -111,7 +113,7 @@ private MapNode buildTreeFromFields(JsonNode node) {
             TreeNode treeNode = buildTreeFromField(entry.getValue());
             namedNodes.add(new NamedNode(name, treeNode));
         });
-        return new MapNode(namedNodes.toArray(new NamedNode[namedNodes.size()]));
+        return new MapNode(namedNodes.toArray(new NamedNode[0]));
     }
 
     private TreeNode buildTreeFromField(JsonNode jsonNode) {
@@ -145,7 +147,7 @@ public TreeNode build(JsonNode jsonNode) {
                 TreeNode treeNode = buildTreeFromField(jsonNode.get(i));
                 treeNodes.add(treeNode);
             }
-            TreeNode[] nodes = treeNodes.toArray(new TreeNode[treeNodes.size()]);
+            TreeNode[] nodes = treeNodes.toArray(new TreeNode[0]);
             return new ArrayNode(nodes);
         }
     }
@@ -161,7 +163,7 @@ public TreeNode build(JsonNode jsonNode) {
                 TreeNode treeNode = buildTreeFromField(entry.getValue());
                 namedNodes.add(new NamedNode(fieldName, treeNode));
             }
-            NamedNode[] nodes = namedNodes.toArray(new NamedNode[namedNodes.size()]);
+            NamedNode[] nodes = namedNodes.toArray(new NamedNode[0]);
             return new MapNode(nodes);
         }
     }

src/main/java/org/seedstack/coffig/provider/PropertiesProvider.java

@@ -85,10 +85,12 @@ public synchronized PropertiesProvider addSource(URL url) {
 
     @Override
     public void fileChanged(Path path) {
+        LOGGER.debug("Configuration file has changed: " + path);
         dirty.set(true);
     }
 
     private MapNode buildTreeFromSource(URL url) {
+        LOGGER.debug("Reading configuration from " + url.toExternalForm());
         try (InputStream inputStream = url.openStream()) {
             MapNode mapNode = new MapNode();
             Properties properties = new Properties();

src/main/java/org/seedstack/coffig/provider/SystemPropertiesProvider.java

@@ -11,10 +11,15 @@
 import org.seedstack.coffig.node.MapNode;
 import org.seedstack.coffig.node.NamedNode;
 import org.seedstack.coffig.spi.ConfigurationProvider;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 public class SystemPropertiesProvider implements ConfigurationProvider {
+    private static final Logger LOGGER = LoggerFactory.getLogger(SystemPropertiesProvider.class);
+
     @Override
     public MapNode provide() {
+        LOGGER.debug("Reading configuration from system properties");
         return new MapNode(System.getProperties().entrySet().stream()
                 .map(e -> new NamedNode((String) e.getKey(), (String) e.getValue()))
                 .toArray(NamedNode[]::new));