From eb32071fe9f76761f2180d808f02bfb81075f59d Mon Sep 17 00:00:00 2001 From: Christopher Dilks Date: Tue, 6 Jan 2026 09:03:21 -0500 Subject: [PATCH] fix: doubled log messages when `SplitLogManager.configureHandlers` is called twice Some classes call `SplitLogManager.configureHandlers` to make the prefix quieter, _e.g._, `DatabaseConstantProvider`; this was causing log messages to double. This PR fixes the issue, and adjusts the formatting a bit to be more consistent with the database classes' log messages. --- .../src/main/java/org/jlab/logging/SplitLogManager.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/common-tools/clas-logging/src/main/java/org/jlab/logging/SplitLogManager.java b/common-tools/clas-logging/src/main/java/org/jlab/logging/SplitLogManager.java index fb8e7047c2..8ade3eba96 100644 --- a/common-tools/clas-logging/src/main/java/org/jlab/logging/SplitLogManager.java +++ b/common-tools/clas-logging/src/main/java/org/jlab/logging/SplitLogManager.java @@ -45,13 +45,15 @@ public static void configureHandlers(Logger logger, boolean includePrefix) { // clear handlers logger.setUseParentHandlers(false); + for(var handler : logger.getHandlers()) + logger.removeHandler(handler); // log message formatting if(includePrefix) // "[source] level: message throwable_backtrace\n" System.setProperty( "java.util.logging.SimpleFormatter.format", - "[" + logger.getName().replaceAll(".*\\.","") + "] %4$s: %5$s%6$s%n"); + "%4$s: [" + logger.getName().replaceAll(".*\\.","") + "] %5$s%6$s%n"); else // "level: message throwable_backtrace\n" System.setProperty(