Excavator: Migrate to the logsafe SafeLogger API

palantir · Jan 4, 2025 · 7462631 · 7462631
1 parent 690f800
commit 7462631
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/...dea-plugin/src/main/java/com/palantir/gradle/versions/intellij/CompletionRefreshUtil.java b/...dea-plugin/src/main/java/com/palantir/gradle/versions/intellij/CompletionRefreshUtil.java
@@ -21,13 +21,15 @@
 import com.intellij.codeInsight.completion.CompletionProcess;
 import com.intellij.codeInsight.completion.CompletionService;
 import com.intellij.openapi.application.ApplicationManager;
+import com.palantir.logsafe.logger.SafeLogger;
+import com.palantir.logsafe.logger.SafeLoggerFactory;
 import java.lang.reflect.InvocationTargetException;
 import java.util.function.Supplier;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public final class CompletionRefreshUtil {
-    private static final Logger log = LoggerFactory.getLogger(CompletionRefreshUtil.class);
+    private static final SafeLogger log = SafeLoggerFactory.get(CompletionRefreshUtil.class);
 
     public static Supplier<Void> refreshOnceSupplier() {
         return Suppliers.memoize(() -> {

diff --git a/.../src/main/java/com/palantir/gradle/versions/intellij/LoadCacheOnGradleProjectRefresh.java b/.../src/main/java/com/palantir/gradle/versions/intellij/LoadCacheOnGradleProjectRefresh.java
@@ -20,12 +20,14 @@
 import com.intellij.openapi.externalSystem.model.task.ExternalSystemTaskNotificationEvent;
 import com.intellij.openapi.externalSystem.model.task.ExternalSystemTaskNotificationListener;
 import com.intellij.openapi.externalSystem.model.task.ExternalSystemTaskType;
+import com.palantir.logsafe.logger.SafeLogger;
+import com.palantir.logsafe.logger.SafeLoggerFactory;
 import org.jetbrains.plugins.gradle.util.GradleConstants;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public class LoadCacheOnGradleProjectRefresh implements ExternalSystemTaskNotificationListener {
-    private static final Logger log = LoggerFactory.getLogger(LoadCacheOnGradleProjectRefresh.class);
+    private static final SafeLogger log = SafeLoggerFactory.get(LoadCacheOnGradleProjectRefresh.class);
 
     @Override
     public final void onSuccess(ExternalSystemTaskId id) {

diff --git a/...ons-idea-plugin/src/main/java/com/palantir/gradle/versions/intellij/RepositoryLoader.java b/...ons-idea-plugin/src/main/java/com/palantir/gradle/versions/intellij/RepositoryLoader.java
@@ -25,6 +25,8 @@
 import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlProperty;
 import com.fasterxml.jackson.datatype.guava.GuavaModule;
 import com.intellij.openapi.project.Project;
+import com.palantir.logsafe.logger.SafeLogger;
+import com.palantir.logsafe.logger.SafeLoggerFactory;
 import java.io.File;
 import java.io.IOException;
 import java.util.Comparator;
@@ -39,7 +41,7 @@
 import org.slf4j.LoggerFactory;
 
 public final class RepositoryLoader {
-    private static final Logger log = LoggerFactory.getLogger(RepositoryLoader.class);
+    private static final SafeLogger log = SafeLoggerFactory.get(RepositoryLoader.class);
 
     private static final ObjectMapper XML_MAPPER = new XmlMapper().registerModule(new GuavaModule());
     private static final String MAVEN_REPOSITORIES_FILE_NAME = ".idea/gcv-maven-repositories.xml";