From c05d1eb916d2f57a33bdcac71a611e8a3061cc21 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bal=C3=A1zs=20Sz=C3=BCcs?= Date: Sat, 15 Nov 2025 16:31:10 +0100 Subject: [PATCH] feat(logging): replace hardcoded "ERROR" with constant for improved maintainability MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Balázs Szücs --- .../java/stirling/software/common/util/ProcessExecutor.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/common/src/main/java/stirling/software/common/util/ProcessExecutor.java b/app/common/src/main/java/stirling/software/common/util/ProcessExecutor.java index 823fcf757..d0cbbdc40 100644 --- a/app/common/src/main/java/stirling/software/common/util/ProcessExecutor.java +++ b/app/common/src/main/java/stirling/software/common/util/ProcessExecutor.java @@ -26,6 +26,7 @@ import stirling.software.common.model.ApplicationProperties; public class ProcessExecutor { private static final Map instances = new ConcurrentHashMap<>(); + private static final String ERROR_KEYWORD = "ERROR"; private static ApplicationProperties applicationProperties = new ApplicationProperties(); private final Semaphore semaphore; private final boolean liveUpdates; @@ -207,7 +208,7 @@ public class ProcessExecutor { != null) { errorLines.add(line); if (liveUpdates) { - if (line.toUpperCase().contains("ERROR")) { + if (line.toUpperCase().contains(ERROR_KEYWORD)) { errorDetected.set(true); } if (errorDetected.get()) { @@ -239,7 +240,7 @@ public class ProcessExecutor { != null) { outputLines.add(line); if (liveUpdates) { - if (line.toUpperCase().contains("ERROR")) { + if (line.toUpperCase().contains(ERROR_KEYWORD)) { errorDetected.set(true); } if (errorDetected.get()) {