From b4e496ff7782d6d7535b3af323faac77d6f5d592 Mon Sep 17 00:00:00 2001 From: Harsh Mehta Date: Tue, 9 Dec 2025 22:59:55 +0530 Subject: [PATCH] Fix swallowed I/O exceptions by replacing PrintWriter with (OutputStreamWriter/BufferedWriter) Signed-off-by: Harsh Mehta --- .../assembly/filter/AbstractLineAggregatingHandler.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/maven/plugins/assembly/filter/AbstractLineAggregatingHandler.java b/src/main/java/org/apache/maven/plugins/assembly/filter/AbstractLineAggregatingHandler.java index b20d2eb3..583a7742 100644 --- a/src/main/java/org/apache/maven/plugins/assembly/filter/AbstractLineAggregatingHandler.java +++ b/src/main/java/org/apache/maven/plugins/assembly/filter/AbstractLineAggregatingHandler.java @@ -19,11 +19,11 @@ package org.apache.maven.plugins.assembly.filter; import java.io.BufferedReader; +import java.io.BufferedWriter; import java.io.File; import java.io.IOException; import java.io.InputStreamReader; import java.io.OutputStreamWriter; -import java.io.PrintWriter; import java.nio.file.Files; import java.util.ArrayList; import java.util.HashMap; @@ -75,10 +75,11 @@ void addToArchive(final Archiver archiver) { f = Files.createTempFile("assembly-" + fname, ".tmp").toFile(); f.deleteOnExit(); - try (PrintWriter writer = - new PrintWriter(new OutputStreamWriter(Files.newOutputStream(f.toPath()), getEncoding()))) { + try (BufferedWriter writer = + new BufferedWriter(new OutputStreamWriter(Files.newOutputStream(f.toPath()), getEncoding()))) { for (final String line : entry.getValue()) { - writer.println(line); + writer.write(line); + writer.newLine(); } } } catch (final IOException e) {