You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by el...@apache.org on 2025/05/30 10:52:16 UTC

(maven-javadoc-plugin) 01/01: Remove some log junk

This is an automated email from the ASF dual-hosted git repository.

elharo pushed a commit to branch log
in repository https://gitbox.apache.org/repos/asf/maven-javadoc-plugin.git

commit 113e1fe6a92a04115035966c100b9205d119d799
Author: Elliotte Rusty Harold <el...@ibiblio.org>
AuthorDate: Fri May 30 06:52:05 2025 -0400

    Remove some log junk
---
 .../org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java  | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java b/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
index d3d2b42d..b81183b0 100644
--- a/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
+++ b/src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
@@ -2120,16 +2120,10 @@ public abstract class AbstractJavadocMojo extends AbstractMojo {
         if (applyJavadocSecurityFix) {
             // finally, patch the Javadoc vulnerability in older Javadoc tools (CVE-2013-1571):
             try {
-                final int patched = fixFrameInjectionBug(javadocOutputDirectory, getDocencoding());
-                if (patched > 0) {
-                    getLog().info(String.format(
-                            "Fixed Javadoc frame injection vulnerability (CVE-2013-1571) in %d files.", patched));
-                }
+                fixFrameInjectionBug(javadocOutputDirectory, getDocencoding());
             } catch (IOException e) {
                 throw new MavenReportException("Failed to patch javadocs vulnerability: " + e.getMessage(), e);
             }
-        } else {
-            getLog().info("applying javadoc security fix has been disabled");
         }
     }
 
@@ -2137,7 +2131,7 @@ public abstract class AbstractJavadocMojo extends AbstractMojo {
      * Method to get the files on the specified source paths
      *
      * @param sourcePaths a Collection that contains the paths to the source files
-     * @return a List that contains the specific path for every source file
+     * @return the specific path for every source file
      * @throws MavenReportException {@link MavenReportException} issue while generating report
      */
     protected Map<Path, Collection<String>> getFiles(Collection<Path> sourcePaths) throws MavenReportException {