Fixes #3, enable to read env variables for getProperty

pom.xml

@@ -74,7 +74,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-compiler-plugin</artifactId>
-        <version>3.8.1</version>
+        <version>3.11.0</version>
         <configuration>
           <source>11</source>
           <target>11</target>
@@ -86,7 +86,7 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-surefire-plugin</artifactId>
-        <version>3.0.0-M5</version>
+        <version>3.0.0</version>
         <configuration>
           <trimStackTrace>false</trimStackTrace>
         </configuration>

src/main/minisite/content/jul-integration.adoc

@@ -150,3 +150,5 @@ And finally a configuration using file output instead of standard one:
 .handlers = file
 file.formatter = inline
 ----
+
+TIP: you can set all properties as system properties and also environment variables (in uppercase and dots replaced by underscores).

yupiik-logging-jul/src/main/java/io/yupiik/logging/jul/YupiikLoggers.java

@@ -47,6 +47,7 @@
 import java.util.logging.Handler;
 import java.util.logging.Level;
 import java.util.logging.Logger;
+import java.util.regex.Pattern;
 import java.util.stream.Stream;
 
 import static java.util.Collections.enumeration;
@@ -59,6 +60,8 @@
 //
 // note: it is not a 100% complete impl yet, it mainly targets docker containers for now and some features as config listeners are ignored
 public class YupiikLoggers {
+    private final Pattern posix = Pattern.compile("[^A-Za-z0-9]");
+
     public static class State { // makes it easy to reset at once
         private final ConcurrentMap<Runnable, Runnable> listeners = new ConcurrentHashMap<>();
         private final ConcurrentMap<String, YupiikLogger> loggers = new ConcurrentHashMap<>();
@@ -106,7 +109,9 @@ public YupiikLogger getLoggerOrNull(final String name) {
     }
 
     public String getProperty(final String name) {
-        return System.getProperty(name, state.configuration.get(name));
+        return ofNullable(System.getProperty(name))
+                .or(() -> ofNullable(System.getenv(posix.matcher(name).replaceAll("_").toUpperCase(ROOT))))
+                .orElseGet(() -> state.configuration.get(name));
     }
 
     public Enumeration<String> getLoggerNames() {

yupiik-logging-jul/src/main/java/io/yupiik/logging/jul/handler/LocalFileHandler.java

@@ -471,7 +471,7 @@ protected static String replace(final String str) { // [lang3] would be good but
                 }
 
                 final String propName = str.substring(start + 2, end);
-                String replacement = !propName.isEmpty() ? System.getProperty(propName) : null;
+                String replacement = !propName.isEmpty() ? readValue(propName) : null;
                 if (replacement == null) {
                     replacement = System.getenv(propName);
                 }
@@ -488,6 +488,13 @@ protected static String replace(final String str) { // [lang3] would be good but
         return result;
     }
 
+    private static String readValue(final String propName) {
+        if (propName.startsWith("env.")) {
+            return System.getenv(propName.substring("env.".length()));
+        }
+        return System.getProperty(propName);
+    }
+
     private final class CountingStream extends OutputStream {
         private final OutputStream out;
 

yupiik-logging-jul/src/test/java/io/yupiik/logging/jul/handler/LocalFileHandlerTest.java

@@ -272,6 +272,6 @@ public String format(final LogRecord record) {
     }
 
     private static LocalDate asDate(final Instant instant) {
-        return instant.atZone(ZoneId.systemDefault()).toLocalDate();
+        return instant.atZone(ZoneId.of("UTC")).toLocalDate();
     }
 }