Merge pull request #71 from informatik-ag-ngl/f/improveeed_logging

Improved logging

src/main/java/envoy/client/Client.java

@@ -1,7 +1,9 @@
 package envoy.client;
 
+import java.io.StringWriter;
 import java.util.HashMap;
 import java.util.Map;
+import java.util.logging.Logger;
 
 import javax.ws.rs.client.ClientBuilder;
 import javax.ws.rs.client.Entity;
@@ -11,6 +13,7 @@ import javax.xml.bind.JAXBContext;
 import javax.xml.bind.JAXBException;
 import javax.xml.bind.Marshaller;
 
+import envoy.client.util.EnvoyLog;
 import envoy.exception.EnvoyException;
 import envoy.schema.ObjectFactory;
 import envoy.schema.Sync;
@@ -33,6 +36,8 @@ public class Client {
 	private User			sender, recipient;
 	private boolean			online			= false;
 
+	private static final Logger logger = EnvoyLog.getLogger(Client.class.getSimpleName());
+
 	/**
 	 * Initializes the client. At this state, the client user has yet to be
 	 * initialized, which can be done by calling {@link Client#onlineInit(String)}.
@@ -163,7 +168,9 @@ public class Client {
 			jc = JAXBContext.newInstance("envoy.schema");
 			Marshaller m = jc.createMarshaller();
 			m.setProperty(Marshaller.JAXB_FORMATTED_OUTPUT, true);
-			m.marshal(sync, System.out);
+			StringWriter stringWriter = new StringWriter();
+			m.marshal(sync, stringWriter);
+			logger.fine("Sending sync:\n" + stringWriter.toString());
 		} catch (JAXBException e) {
 			e.printStackTrace();
 		}

src/main/java/envoy/client/Config.java

@@ -32,7 +32,8 @@ public class Config {
 		items.put("port", new ConfigItem<>("port", "p", (input) -> Integer.parseInt(input), null));
 		items.put("localDB", new ConfigItem<>("localDB", "db", (input) -> new File(input), new File("localDB")));
 		items.put("syncTimeout", new ConfigItem<>("syncTimeout", "st", (input) -> Integer.parseInt(input), 1000));
-		items.put("homeDirectory", new ConfigItem<>("homeDirectory", "h", (input) -> new File(input), new File(System.getProperty("user.home"), ".envoy")));
+		items.put("homeDirectory",
+				new ConfigItem<>("homeDirectory", "h", (input) -> new File(input), new File(System.getProperty("user.home"), ".envoy")));
 		items.put("fileLevelBarrier", new ConfigItem<>("fileLevelBarrier", "fb", (input) -> Level.parse(input), Level.CONFIG));
 		items.put("consoleLevelBarrier", new ConfigItem<>("consoleLevelBarrier", "cb", (input) -> Level.parse(input), Level.FINEST));
 	}

src/main/java/envoy/client/LocalDB.java

@@ -1,16 +1,8 @@
 package envoy.client;
 
-import java.io.File;
+import java.io.*;
-import java.io.FileInputStream;
-import java.io.FileOutputStream;
-import java.io.IOException;
-import java.io.ObjectInputStream;
-import java.io.ObjectOutputStream;
 import java.time.Instant;
-import java.util.ArrayList;
+import java.util.*;
-import java.util.HashMap;
-import java.util.List;
-import java.util.Map;
 import java.util.logging.Logger;
 
 import javax.xml.datatype.DatatypeConfigurationException;
@@ -20,11 +12,8 @@ import envoy.client.event.EventBus;
 import envoy.client.event.MessageCreationEvent;
 import envoy.client.util.EnvoyLog;
 import envoy.exception.EnvoyException;
-import envoy.schema.Message;
+import envoy.schema.*;
 import envoy.schema.Message.Metadata.MessageState;
-import envoy.schema.ObjectFactory;
-import envoy.schema.Sync;
-import envoy.schema.User;
 
 /**
  * Project: <strong>envoy-client</strong><br>
@@ -249,10 +238,8 @@ public class LocalDB {
 		// Updating UserStatus of all users in LocalDB
 		for (User user : returnSync.getUsers())
 			for (Chat chat : getChats())
-				if (user.getID() == chat.getRecipient().getID()) {
+				if (user.getID() == chat.getRecipient().getID())
 					chat.getRecipient().setStatus(user.getStatus());
-					logger.info(chat.getRecipient().getStatus().toString());
-				}
 
 		sync.getMessages().clear();
 		sync.getUsers().clear();

src/main/java/envoy/client/util/EnvoyLog.java

@@ -18,11 +18,18 @@ import envoy.client.Config;
 public class EnvoyLog {
 
 	private static FileHandler		fileHandler;
-	private static ConsoleHandler	consoleHandler;
+	private static StreamHandler	consoleHandler;
 
 	static {
+		// Remove default console handler
+		LogManager.getLogManager().reset();
+
+		// Configure log file
 		File logFile = new File(Config.getInstance().getHomeDirectory(), "log/envoy_user.log");
 		logFile.getParentFile().mkdirs();
+
+		// Configure formatting
+		System.setProperty("java.util.logging.SimpleFormatter.format", "[%1$tF %1$tT] [%4$-7s] [%3$s] %5$s %n");
 		SimpleFormatter formatter = new SimpleFormatter();
 
 		try {
@@ -32,7 +39,7 @@ public class EnvoyLog {
 		} catch (SecurityException | IOException e) {
 			e.printStackTrace();
 		}
-		consoleHandler = new ConsoleHandler();
+		consoleHandler = new StreamHandler(System.out, formatter);
 		consoleHandler.setLevel(Config.getInstance().getConsoleLevelBarrier());
 		consoleHandler.setFormatter(formatter);
 	}