(maven-dist-tool) branch master updated: Avoid break build on IOException in check-source-release report

[sjaranowski]

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

sjaranowski pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/maven-dist-tool.git


The following commit(s) were added to refs/heads/master by this push:
     new 06b8119  Avoid break build on IOException in check-source-release 
report
06b8119 is described below

commit 06b81198983074349145b57bcf433baf4f14c119
Author: Slawomir Jaranowski <s.jaranow...@gmail.com>
AuthorDate: Sat Nov 8 15:26:43 2025 +0100

    Avoid break build on IOException in check-source-release report
    
    When IOException occurs we cen continue to generating report instead of 
stop whole build
---
 .../tools/source/DistCheckSourceReleaseReport.java    | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

diff --git 
a/src/main/java/org/apache/maven/dist/tools/source/DistCheckSourceReleaseReport.java
 
b/src/main/java/org/apache/maven/dist/tools/source/DistCheckSourceReleaseReport.java
index 4648a44..8ecf658 100644
--- 
a/src/main/java/org/apache/maven/dist/tools/source/DistCheckSourceReleaseReport.java
+++ 
b/src/main/java/org/apache/maven/dist/tools/source/DistCheckSourceReleaseReport.java
@@ -455,20 +455,22 @@ public class DistCheckSourceReleaseReport extends 
AbstractDistCheckReport {
      * @param cli
      * @param version
      * @return missing files
-     * @throws IOException
      */
-    private List<String> checkDirectoryIndex(String url, ConfigurationLineInfo 
cli, String version, boolean dist)
-            throws IOException {
+    private List<String> checkDirectoryIndex(String url, ConfigurationLineInfo 
cli, String version, boolean dist) {
         Set<String> retrievedFiles = new HashSet<>();
-        Elements links = selectLinks(url);
-        for (Element e : links) {
-            retrievedFiles.add(e.attr("href"));
+        List<String> missingFiles = new ArrayList<>();
+
+        try {
+            Elements links = selectLinks(url);
+            for (Element e : links) {
+                retrievedFiles.add(e.attr("href"));
+            }
+        } catch (IOException e) {
+            missingFiles.add(url + ": " + e.getMessage());
         }
 
         String sourceReleaseFilename = cli.getSourceReleaseFilename(version, 
dist);
 
-        List<String> missingFiles = new ArrayList<>();
-
         // require source release file
         if (!retrievedFiles.contains(sourceReleaseFilename)) {
             missingFiles.add(sourceReleaseFilename);
@@ -491,7 +493,6 @@ public class DistCheckSourceReleaseReport extends 
AbstractDistCheckReport {
             }
             if (error) {
                 getLog().warn("==> when reading " + url + " got following 
hrefs: " + retrievedFiles);
-                getLog().warn(url + " = " + read(url));
             }
         }