commit 3a9a4dfbb8e02e5eefbec8bccaa8df06c1331f34
author Fabio Ponciroli <ponch78@gmail.com> Thu Apr 21 09:02:55 2022 +0200
committer Fabio Ponciroli <ponch78@gmail.com> Thu Apr 21 09:02:55 2022 +0200
tree bbc9015263219fcab50abbe3afc825e87808d402
parent b5317a69800104d6031e6dab5d34901bc784359d

Fix logging arguments

Bug: Issue 15863
Change-Id: I424297a9bd679596fb3e007b43134496f04ded5a

src/main/java/com/googlesource/gerrit/plugins/websession/broker/BrokerBasedWebSessionCache.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/websession/broker/BrokerBasedWebSessionCache.java b/src/main/java/com/googlesource/gerrit/plugins/websession/broker/BrokerBasedWebSessionCache.java
index a0d9218..d7255e2 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/websession/broker/BrokerBasedWebSessionCache.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/websession/broker/BrokerBasedWebSessionCache.java
@@ -88,7 +88,7 @@
 
   protected void processMessage(Event message) {
     if (!WebSessionEvent.TYPE.equals(message.getType())) {
-      logger.atWarning().log("Skipping web session message of unknown type:{}", message.getType());
+      logger.atWarning().log("Skipping web session message of unknown type: %s", message.getType());
       return;
     }
 
@@ -107,7 +107,7 @@
           }
 
         } catch (IOException | ClassNotFoundException e) {
-          logger.atSevere().withCause(e).log("Malformed event '%s': [Exception: %s]", message);
+          logger.atSevere().withCause(e).log("Malformed event '%s'", message);
         }
         break;
       case REMOVE:
@@ -117,7 +117,7 @@
         break;
       default:
         logger.atWarning().log(
-            "Skipping web session message of unknown operation type:{}", event.operation);
+            "Skipping web session message of unknown operation type: %s", event.operation);
         break;
     }
   }
@@ -282,7 +282,7 @@
             MoreExecutors.directExecutor());
       } catch (IOException e) {
         logger.atSevere().withCause(e).log(
-            "Cannot serialize event for account id '%s': [Exception: %s]", value.getAccountId());
+            "Cannot serialize event for account id '%s'", value.getAccountId());
       }
     }
   }