From b4edbe29869d40a8f283a876e9d1b368952f1d9b Mon Sep 17 00:00:00 2001
From: Manfred Karrer <mk@nucleo.io>
Date: Sat, 23 Apr 2016 02:51:01 +0200
Subject: [PATCH] Improve logging

---
 common/src/main/java/io/bitsquare/app/Log.java            | 8 ++++++--
 core/src/main/java/io/bitsquare/trade/offer/Offer.java    | 2 +-
 gui/src/main/java/io/bitsquare/app/BitsquareApp.java      | 4 +++-
 .../main/java/io/bitsquare/p2p/network/Connection.java    | 8 ++++++--
 4 files changed, 16 insertions(+), 6 deletions(-)

diff --git a/common/src/main/java/io/bitsquare/app/Log.java b/common/src/main/java/io/bitsquare/app/Log.java
index 810da81353a..5c08e9db0ab 100644
--- a/common/src/main/java/io/bitsquare/app/Log.java
+++ b/common/src/main/java/io/bitsquare/app/Log.java
@@ -30,7 +30,7 @@ public class Log {
     private static SizeBasedTriggeringPolicy triggeringPolicy;
     private static Logger logbackLogger;
 
-    public static void setup(String fileName, boolean useDetailedLogging) {
+    public static void setup(String fileName) {
         LoggerContext loggerContext = (LoggerContext) LoggerFactory.getILoggerFactory();
 
         RollingFileAppender appender = new RollingFileAppender();
@@ -60,8 +60,8 @@ public static void setup(String fileName, boolean useDetailedLogging) {
         appender.start();
 
         logbackLogger = loggerContext.getLogger(org.slf4j.Logger.ROOT_LOGGER_NAME);
-        logbackLogger.setLevel(useDetailedLogging ? Level.TRACE : Level.WARN);
         logbackLogger.addAppender(appender);
+        logbackLogger.setLevel(Level.INFO);
 
         // log errors in separate file
         // not working as expected still.... damn logback...
@@ -80,6 +80,10 @@ public static void setup(String fileName, boolean useDetailedLogging) {
         logbackLogger.addAppender(errorAppender);*/
     }
 
+    public static void setLevel(boolean useDetailedLogging) {
+        logbackLogger.setLevel(useDetailedLogging ? Level.TRACE : Level.WARN);
+    }
+
     public static void traceCall() {
         if (LoggerFactory.getLogger(Log.class).isTraceEnabled()) {
             StackTraceElement stackTraceElement = new Throwable().getStackTrace()[1];
diff --git a/core/src/main/java/io/bitsquare/trade/offer/Offer.java b/core/src/main/java/io/bitsquare/trade/offer/Offer.java
index 39e862d40cb..e912c0d32ce 100644
--- a/core/src/main/java/io/bitsquare/trade/offer/Offer.java
+++ b/core/src/main/java/io/bitsquare/trade/offer/Offer.java
@@ -371,7 +371,7 @@ public Fiat getPrice() {
                     return null;
                 }
             } else {
-                log.warn("We don't have a market price.\n" +
+                log.debug("We don't have a market price.\n" +
                         "That case could only happen if you don't have a price feed.");
                 return null;
             }
diff --git a/gui/src/main/java/io/bitsquare/app/BitsquareApp.java b/gui/src/main/java/io/bitsquare/app/BitsquareApp.java
index dae6210dd3e..9d668907747 100644
--- a/gui/src/main/java/io/bitsquare/app/BitsquareApp.java
+++ b/gui/src/main/java/io/bitsquare/app/BitsquareApp.java
@@ -99,10 +99,12 @@ public static void setEnvironment(Environment env) {
     @Override
     public void start(Stage primaryStage) throws IOException {
         String logPath = Paths.get(env.getProperty(BitsquareEnvironment.APP_DATA_DIR_KEY), "bitsquare").toString();
-        Log.setup(logPath, !IS_RELEASE_VERSION);
+
+        Log.setup(logPath);
         log.info("Log files under: " + logPath);
         Version.printVersion();
         Utilities.printSysInfo();
+        Log.setLevel(!IS_RELEASE_VERSION);
 
         UserThread.setExecutor(Platform::runLater);
         UserThread.setTimerClass(UITimer.class);
diff --git a/network/src/main/java/io/bitsquare/p2p/network/Connection.java b/network/src/main/java/io/bitsquare/p2p/network/Connection.java
index e76af68c10f..d75679e7640 100644
--- a/network/src/main/java/io/bitsquare/p2p/network/Connection.java
+++ b/network/src/main/java/io/bitsquare/p2p/network/Connection.java
@@ -542,8 +542,12 @@ public void handleConnectionException(Throwable e) {
             } else {
                 // TODO sometimes we get StreamCorruptedException, OptionalDataException, IllegalStateException
                 closeConnectionReason = CloseConnectionReason.UNKNOWN_EXCEPTION;
-                log.warn("Unknown reason for exception at socket {}\n\tconnection={}\n\tException=",
-                        socket.toString(), this, e.toString());
+                log.warn("Unknown reason for exception at socket {}\n\t" +
+                                "connection={}\n\t" +
+                                "Exception=",
+                        socket.toString(),
+                        this,
+                        e.toString());
                 e.printStackTrace();
             }