commit 88564138366e9e1691c3a762c9f46af955747934
author Jorge Ruesga <j.ruesga.criado@gmail.com> Thu Dec 15 00:17:39 2016 +0100
committer Jorge Ruesga <j.ruesga.criado@gmail.com> Thu Dec 15 00:17:39 2016 +0100
tree 238393358ce105948f89d01a778d58e2c38ba900
parent efb3ff472b6eba2f521b91ebe801af42fa043666

Fix bad debug formatting

Replace wrong debug formats ([%]) and add a trace to allow debug
the fcm response.

Change-Id: Ifb38456449ee930e59b9782d1bd1ca488d0bdc84

src/main/java/com/ruesga/gerrit/plugins/fcm/workers/FcmUploaderWorker.java

diff --git a/src/main/java/com/ruesga/gerrit/plugins/fcm/workers/FcmUploaderWorker.java b/src/main/java/com/ruesga/gerrit/plugins/fcm/workers/FcmUploaderWorker.java
index c4b00ab..b03aa65 100644
--- a/src/main/java/com/ruesga/gerrit/plugins/fcm/workers/FcmUploaderWorker.java
+++ b/src/main/java/com/ruesga/gerrit/plugins/fcm/workers/FcmUploaderWorker.java
@@ -119,7 +119,7 @@
             String data = gson.toJson(submit.request);
             if (log.isDebugEnabled()) {
                 log.debug(String.format(
-                        "[%] Sending fcm notification: %s", pluginName, data));
+                        "[%s] Sending fcm notification: %s", pluginName, data));
             }
 
             URL url = new URL(config.serverUrl);
@@ -164,6 +164,11 @@
                 }
 
                 // Process the server response
+                if (log.isDebugEnabled()) {
+                    log.debug(String.format(
+                        "[%s] fcm response: %s",
+                        pluginName, response.toString()));
+                }
                 processResponse(conn, submit, gson.fromJson(
                         response.toString(), FcmResponseInfo.class));
 
@@ -223,7 +228,7 @@
                 case "NotRegistered":
                     // Remove this client from the database
                     if (log.isDebugEnabled()) {
-                        log.debug("[%] %d - %s - %s is not registered. " +
+                        log.debug("[%s] %d - %s - %s is not registered. " +
                                 "Remove from db.",
                                 pluginName, submit.accountId,
                                 submit.device,
@@ -272,7 +277,7 @@
         }
 
         if (log.isDebugEnabled()) {
-            log.debug("[%] Retry fcm notification to %s after %d seconds",
+            log.debug("[%s] Retry fcm notification to %s after %d seconds",
                     pluginName, submit.request.to, retryAfter);
         }
         this.delayedExecutor.schedule(new Runnable() {