From b88b13e64ece06dec788f8160df07f2b58a497dc Mon Sep 17 00:00:00 2001
From: Roberto Cella <rob.uniud@gmail.com>
Date: Tue, 21 Jul 2026 10:35:46 +0200
Subject: [PATCH 1/3] Separate standard output and error output in
 ProcessHelper

---
 .../stickerify/media/MediaHelper.java         |  2 +-
 .../stickerify/process/ProcessHelper.java     | 39 ++++++++++++++-----
 2 files changed, 30 insertions(+), 11 deletions(-)

diff --git a/src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java b/src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java
index 9932f8f0..5ba78ee7 100644
--- a/src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java
+++ b/src/main/java/com/github/stickerifier/stickerify/media/MediaHelper.java
@@ -186,7 +186,7 @@ static MultimediaInfo retrieveMultimediaInfo(File file) throws MediaException, I
 		var command = new String[] {
 				"ffprobe",
 				"-hide_banner",
-				"-v", "quiet",
+				"-v", "error",
 				"-print_format", "json",
 				"-show_format",
 				"-show_streams",
diff --git a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
index be54c530..9e3ff39f 100644
--- a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
+++ b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
@@ -22,7 +22,7 @@ public final class ProcessHelper {
 	 * environment variable (defaults to 4).
 	 *
 	 * @param command the command to be executed
-	 * @return the merged stdout/stderr of the command
+	 * @return the standard output of the command
 	 * @throws ProcessException either if:
 	 * <ul>
 	 *     <li>the command was unsuccessful
@@ -35,30 +35,49 @@ public final class ProcessHelper {
 	public static String executeCommand(final String... command) throws ProcessException, InterruptedException {
 		SEMAPHORE.acquire();
 
-		try (var process = new ProcessBuilder(command).redirectErrorStream(true).start()) {
-			var output = new StringJoiner("\n");
-			var readerThread = Thread.ofVirtual().start(() -> {
+		try (var process = new ProcessBuilder(command).start()) {
+			var standardOutput = new StringJoiner("\n");
+			var outputThread = Thread.ofVirtual().start(() -> {
 				try (var reader = process.inputReader(UTF_8)) {
-					reader.lines().forEach(output::add);
+					reader.lines().forEach(standardOutput::add);
 				} catch (IOException e) {
 					LOGGER.at(Level.ERROR).setCause(e).log("Error while closing process output reader");
 				}
 			});
 
+			var standardError = new StringJoiner("\n");
+			var errorThread = Thread.ofVirtual().start(() -> {
+				try (var reader = process.errorReader(UTF_8)) {
+					reader.lines().forEach(standardError::add);
+				} catch (IOException e) {
+					LOGGER.at(Level.ERROR).setCause(e).log("Error while closing process error reader");
+				}
+			});
+
 			var finished = process.waitFor(1, TimeUnit.MINUTES);
 			if (!finished) {
 				process.destroyForcibly();
-				readerThread.join();
-				throw new ProcessException("The command {} timed out after 1m\n{}", command[0], output.toString());
+				outputThread.join();
+				errorThread.join();
+				throw new ProcessException("The command {} timed out after 1m\nStandard output: {}\nStandard error: {}",
+						command[0],
+						standardOutput.toString(),
+						standardError.toString());
 			}
 
-			readerThread.join();
+			outputThread.join();
+			errorThread.join();
+
 			var exitCode = process.exitValue();
 			if (exitCode != 0) {
-				throw new ProcessException("The command {} exited with code {}\n{}", command[0], exitCode, output.toString());
+				throw new ProcessException("The command {} exited with code {}\nStandard output: {}\nStandard error: {}",
+						command[0],
+						exitCode,
+						standardOutput.toString(),
+						standardError.toString());
 			}
 
-			return output.toString();
+			return standardOutput.toString();
 		} catch (IOException e) {
 			throw new ProcessException(e);
 		} finally {

From e5e1b3341997d25e673acef454873b11a481aae0 Mon Sep 17 00:00:00 2001
From: Roberto Cella <rob.uniud@gmail.com>
Date: Tue, 21 Jul 2026 10:52:05 +0200
Subject: [PATCH 2/3] Simplify output information when an error occurs

---
 .../stickerify/process/ProcessHelper.java           | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
index 9e3ff39f..3677372e 100644
--- a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
+++ b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
@@ -59,10 +59,8 @@ public static String executeCommand(final String... command) throws ProcessExcep
 				process.destroyForcibly();
 				outputThread.join();
 				errorThread.join();
-				throw new ProcessException("The command {} timed out after 1m\nStandard output: {}\nStandard error: {}",
-						command[0],
-						standardOutput.toString(),
-						standardError.toString());
+				LOGGER.at(Level.WARN).log("The command {} timed out after 1m: {}", command[0], standardError.toString());
+				throw new ProcessException("The command {} timed out after 1m", command[0]);
 			}
 
 			outputThread.join();
@@ -70,11 +68,8 @@ public static String executeCommand(final String... command) throws ProcessExcep
 
 			var exitCode = process.exitValue();
 			if (exitCode != 0) {
-				throw new ProcessException("The command {} exited with code {}\nStandard output: {}\nStandard error: {}",
-						command[0],
-						exitCode,
-						standardOutput.toString(),
-						standardError.toString());
+				LOGGER.at(Level.WARN).log("The command {} exited with code {}: {}", command[0], exitCode, standardError.toString());
+				throw new ProcessException("The command {} exited with code {}", command[0], exitCode);
 			}
 
 			return standardOutput.toString();

From 949722d4b27d669f01751134b8352bfefd9cbc0e Mon Sep 17 00:00:00 2001
From: Roberto Cella <rob.uniud@gmail.com>
Date: Tue, 21 Jul 2026 11:15:51 +0200
Subject: [PATCH 3/3] Add logic to handle silent failures when using process
 readers

---
 .../stickerifier/stickerify/process/ProcessHelper.java   | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
index 3677372e..5aee46ce 100644
--- a/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
+++ b/src/main/java/com/github/stickerifier/stickerify/process/ProcessHelper.java
@@ -7,6 +7,7 @@
 import org.slf4j.event.Level;
 
 import java.io.IOException;
+import java.io.UncheckedIOException;
 import java.util.StringJoiner;
 import java.util.concurrent.Semaphore;
 import java.util.concurrent.TimeUnit;
@@ -40,8 +41,8 @@ public static String executeCommand(final String... command) throws ProcessExcep
 			var outputThread = Thread.ofVirtual().start(() -> {
 				try (var reader = process.inputReader(UTF_8)) {
 					reader.lines().forEach(standardOutput::add);
-				} catch (IOException e) {
-					LOGGER.at(Level.ERROR).setCause(e).log("Error while closing process output reader");
+				} catch (IOException | UncheckedIOException e) {
+					LOGGER.at(Level.ERROR).setCause(e).log("An error occurred using process output reader");
 				}
 			});
 
@@ -49,8 +50,8 @@ public static String executeCommand(final String... command) throws ProcessExcep
 			var errorThread = Thread.ofVirtual().start(() -> {
 				try (var reader = process.errorReader(UTF_8)) {
 					reader.lines().forEach(standardError::add);
-				} catch (IOException e) {
-					LOGGER.at(Level.ERROR).setCause(e).log("Error while closing process error reader");
+				} catch (IOException | UncheckedIOException e) {
+					LOGGER.at(Level.ERROR).setCause(e).log("An error occurred using process error reader");
 				}
 			});