Fix logging arguments Bug: Issue 15863 Change-Id: I424297a9bd679596fb3e007b43134496f04ded5a
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()); } } }