commit 73df3d51e2dacbc214531a8b0ce3767dc14ca476
author Darius Jokilehto <dariusjokilehto+os@gmail.com> Fri Jun 02 14:20:41 2023 +0100
committer Darius Jokilehto <dariusjokilehto+os@gmail.com> Fri Jun 02 14:20:41 2023 +0100
tree 3d14731ff195762dd5fb5485bdd71d8183328c38
parent 1f4c4127e72473a875ef31f4e25eb623eb478c09

Fix flogger compile time errors

Change-Id: I7838847506b164734cd83f6dcd76cc9e68c23015

src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsLocksContext.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsLocksContext.java b/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsLocksContext.java
index 5e4094c..ba2a9af 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsLocksContext.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsLocksContext.java
@@ -98,7 +98,7 @@
   }
 
   void sendError(int status, Error error) throws IOException {
-    log.atSevere().log(error.message);
+    log.atSevere().log("%s", error.message);
     res.setStatus(status);
     gson.toJson(error, getWriter());
     getWriter().flush();

src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsProjectLocks.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsProjectLocks.java b/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsProjectLocks.java
index 8c0345c..3b3daef 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsProjectLocks.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/lfs/locks/LfsProjectLocks.java
@@ -108,7 +108,7 @@
           String.format(
               "Locking path [%s] in project %s failed with error %s",
               input.path, project, e.getMessage());
-      log.atWarning().log(error);
+      log.atWarning().log("%s", error);
       throw new LfsException(error);
     }
 
@@ -120,13 +120,13 @@
             String.format(
                 "Locking path [%s] in project %s failed during write with error %s",
                 input.path, project, e.getMessage());
-        log.atWarning().log(error);
+        log.atWarning().log("%s", error);
         throw new LfsException(error);
       }
       if (!fileLock.commit()) {
         String error =
             String.format("Committing lock to path [%s] in project %s failed", input.path, project);
-        log.atWarning().log(error);
+        log.atWarning().log("%s", error);
         throw new LfsException(error);
       }
       // put lock object to cache while file lock is being hold so that
@@ -148,7 +148,7 @@
         String error =
             String.format(
                 "Deleting lock on path [%s] in project %s is not possible", lock.path, project);
-        log.atWarning().log(error);
+        log.atWarning().log("%s", error);
         throw new LfsException(error);
       }
     } catch (IOException e) {
@@ -156,7 +156,7 @@
           String.format(
               "Getting lock on path [%s] in project %s failed with error %s",
               lock.path, project, e.getMessage());
-      log.atWarning().log(error);
+      log.atWarning().log("%s", error);
       throw new LfsException(error);
     }
 
@@ -168,7 +168,7 @@
           String.format(
               "Deleting lock on path [%s] in project %s failed with error %s",
               lock.path, project, e.getMessage());
-      log.atWarning().log(error);
+      log.atWarning().log("%s", error);
       throw new LfsException(error);
     } finally {
       fileLock.unlock();