This is an automated email from the ASF dual-hosted git repository. kwin pushed a commit to branch feature/throw-uoe-for-multipagesinkfactory-createSink in repository https://gitbox.apache.org/repos/asf/maven-site-plugin.git
commit 6c1d52ba598e24ce3a8c98227a6e8f861a0a5e57 Author: Konrad Windszus <[email protected]> AuthorDate: Fri Feb 20 20:29:37 2026 +0100 Throw UOE for unsupported MultiPageSinkFactory.createSink(...) overloads This prevent NPE which was previously experienced due to non-allowed null value being returned. This closes #1231 --- .../maven/plugins/site/render/ReportDocumentRenderer.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java b/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java index 26f04ec9..a33d4ffc 100644 --- a/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java +++ b/src/main/java/org/apache/maven/plugins/site/render/ReportDocumentRenderer.java @@ -147,20 +147,20 @@ public class ReportDocumentRenderer implements DocumentRenderer { @Override public Sink createSink(File arg0, String arg1, String arg2) throws IOException { - // Not used - return null; + throw new UnsupportedOperationException( + "Only createSink(File, String) is supported by MultiPageSinkFactory"); } @Override public Sink createSink(OutputStream arg0) throws IOException { - // Not used - return null; + throw new UnsupportedOperationException( + "Only createSink(File, String) is supported by MultiPageSinkFactory"); } @Override public Sink createSink(OutputStream arg0, String arg1) throws IOException { - // Not used - return null; + throw new UnsupportedOperationException( + "Only createSink(File, String) is supported by MultiPageSinkFactory"); } public List<MultiPageSubSink> sinks() {
