Fixed logger for production.

xola · Jan 5, 2017 · e0a6f5f · e0a6f5f
1 parent 8933df8
commit e0a6f5f
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/app/controllers/IncomingDataController.java b/app/controllers/IncomingDataController.java
@@ -75,8 +75,10 @@ public CompletionStage<Result> index() {
                 case Event.PLUGIN_INSTALL:
                     log.debug("Installation event received");
                     return complete(executeInstallationEvents(event, json, messages));
+                case Event.PLUGIN_UNINSTALL:
+                    log.warn("Ignoring event {}.", event);
                 default:
-                    log.warn("Ignoring event {}.");
+                    log.warn("Ignoring event {}.", event);
                     return complete(badRequest(ErrorUtil.toJson(BAD_REQUEST, messages.at(MessageKey.NOT_SUBSCRIBED))));
             }
         } else {
@@ -132,7 +134,7 @@ private Result executeInstallationEvents(String event, JsonNode json, Messages m
                 return updateHelper.updateConfiguration(data, messages);
             }
         } catch (Exception e) {
-            log.debug("Missing or invalid data object.", e);
+            log.error("Missing or invalid data object.", e);
             Map<String, String> errorMessages = new HashMap<>();
             errorMessages.put("payload.data", messages.at(MessageKey.MISSING_PAYLOAD_DATA));
             return badRequest(ErrorUtil.toJson(BAD_REQUEST, messages.at(MessageKey.INVALID_JSON), errorMessages));

diff --git a/conf/logback-prod.xml b/conf/logback-prod.xml
@@ -2,9 +2,9 @@
 <configuration>
 
     <logger name="play" level="INFO"/>
-    <logger name="application" level="INFO"/>
-    <logger name="controllers" level="INFO"/>
-    <logger name="daos" level="INFO"/>
+    <logger name="application" level="debug"/>
+    <logger name="controllers" level="debug"/>
+    <logger name="daos" level="debug"/>
 
 
     <!-- Off these ones as they are annoying, and anyway we manage configuration ourselves -->