improve logging, update logging config

src/main/java/name/maxdeliso/teflon/Arguments.java

@@ -8,7 +8,7 @@
 @SuppressWarnings({"unused"})
 class Arguments {
     @Parameter(names = {"--mode", "-m"}, description = "the run mode, one of L[ist interfaces] or R[un]")
-    private String mode;
+    private String mode = "R";
 
     @Parameter(names = {"--help", "-h"}, help = true)
     private boolean help;

src/main/java/name/maxdeliso/teflon/Teflon.java

@@ -26,7 +26,9 @@ public static void main(String[] args) {
         final Optional<String> modeOpt = ofNullable(arguments.getMode());
 
         if (arguments.isHelp() || modeOpt.isEmpty()) {
-            jc.usage();
+            final StringBuilder sb = new StringBuilder();
+            jc.usage(sb);
+            LOG.error(sb.toString());
             return;
         }
 

src/main/java/name/maxdeliso/teflon/config/JsonConfigLoader.java

@@ -22,7 +22,7 @@ public Optional<Config> loadFromFile(String path) {
         try (final var fileReader = new FileReader(path)) {
             return Optional.ofNullable(gson.fromJson(fileReader, Config.class));
         } catch (final IOException ioe) {
-            LOG.warn("failed to load config", ioe);
+            LOG.error("failed to load config", ioe);
             return Optional.empty();
         }
     }

src/main/java/name/maxdeliso/teflon/di/TeflonModule.java

@@ -12,7 +12,10 @@
 import name.maxdeliso.teflon.data.Message;
 import name.maxdeliso.teflon.net.NetSelector;
 import name.maxdeliso.teflon.ui.MainFrame;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
+import javax.inject.Singleton;
 import java.net.InetAddress;
 import java.net.NetworkInterface;
 import java.net.SocketException;
@@ -21,10 +24,10 @@
 import java.util.concurrent.BlockingQueue;
 import java.util.concurrent.LinkedBlockingQueue;
 
-import javax.inject.Singleton;
-
 @Module
 public class TeflonModule {
+    private static final Logger LOG = LoggerFactory.getLogger(TeflonModule.class);
+
     private static final String CONFIG_PATH = "teflon.json";
 
     @Provides
@@ -48,10 +51,12 @@ JsonMessageMarshaller provideJSONMessageMarshaller(final Gson gson) {
     @Provides
     @Singleton
     Config provideConfig(final ConfigLoader configLoader) {
-        return configLoader
-                .loadFromFile(CONFIG_PATH)
-                .orElseThrow(() -> new IllegalArgumentException(
-                        "failed to locate and load config file at: " + CONFIG_PATH));
+        return configLoader.loadFromFile(CONFIG_PATH)
+                .orElseThrow(() -> {
+                    final String errorMesage = "failed to locate config file at: " + CONFIG_PATH;
+                    LOG.error(errorMesage);
+                    return new IllegalArgumentException(errorMesage);
+                });
     }
 
     @Provides
@@ -80,6 +85,7 @@ InetAddress provideInetAddress(final Config config) {
             // NOTE: this causes a DNS Query to run
             return InetAddress.getByName(config.getMulticastGroup());
         } catch (UnknownHostException uhe) {
+            LOG.error("failed to process multicast group from config", uhe);
             throw new RuntimeException(uhe);
         }
     }
@@ -92,6 +98,7 @@ NetworkInterface provideNetworkInterface(final Config config) {
             // NOTE: this invokes some platform specific code
             return NetworkInterface.getByName(config.getInterfaceName());
         } catch (SocketException se) {
+            LOG.error("failed to process interface name from config", se);
             throw new RuntimeException(se);
         }
     }

src/main/resources/log4j2.xml

@@ -17,7 +17,7 @@
         </RollingFile>
     </Appenders>
     <Loggers>
-        <Logger level="DEBUG" name="name.maxdeliso.teflon" additivity="false">
+        <Logger level="DEBUG" name="name.maxdeliso.teflon" additivity="true">
             <AppenderRef ref="teflon" />
         </Logger>
         <Root level="INFO">