(maven-resolver) branch master updated: WarnChecksumPolicy: proceed on no checksums (#1922)

[cstamas]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new caf6d498f WarnChecksumPolicy: proceed on no checksums (#1922)
caf6d498f is described below

commit caf6d498f4cf7461530493d691d9b631c8ea6026
Author: Gerd Aschemann <git...@aschemann.net>
AuthorDate: Sun Jun 14 14:57:38 2026 +0200

    WarnChecksumPolicy: proceed on no checksums (#1922)
    
    Override AbstractChecksumPolicy.onNoMoreChecksums() in
    WarnChecksumPolicy to log a warning and return instead of
    inheriting the unconditional throw. Brings the policy in
    line with its name: warn on issues but proceed.
    
    FailChecksumPolicy unchanged.
    
    Refs GH-1920.
---
 .../java/org/eclipse/aether/internal/impl/WarnChecksumPolicy.java | 8 ++++++++
 .../org/eclipse/aether/internal/impl/WarnChecksumPolicyTest.java  | 7 +------
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/WarnChecksumPolicy.java
 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/WarnChecksumPolicy.java
index 0670254d1..f9c47d7d3 100644
--- 
a/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/WarnChecksumPolicy.java
+++ 
b/maven-resolver-impl/src/main/java/org/eclipse/aether/internal/impl/WarnChecksumPolicy.java
@@ -39,4 +39,12 @@ final class WarnChecksumPolicy extends 
AbstractChecksumPolicy {
                 exception);
         return true;
     }
+
+    @Override
+    public void onNoMoreChecksums() {
+        logger.warn(
+                "Could not validate integrity of download from {}{}: no 
checksums available",
+                resource.getRepositoryUrl(),
+                resource.getResourceName());
+    }
 }
diff --git 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/WarnChecksumPolicyTest.java
 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/WarnChecksumPolicyTest.java
index db657b11c..ac8f3764f 100644
--- 
a/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/WarnChecksumPolicyTest.java
+++ 
b/maven-resolver-impl/src/test/java/org/eclipse/aether/internal/impl/WarnChecksumPolicyTest.java
@@ -79,11 +79,6 @@ public class WarnChecksumPolicyTest {
 
     @Test
     void testOnNoMoreChecksums() {
-        try {
-            policy.onNoMoreChecksums();
-            fail("No exception");
-        } catch (ChecksumFailureException e) {
-            assertTrue(e.getMessage().contains("no checksums available"));
-        }
+        assertDoesNotThrow(policy::onNoMoreChecksums);
     }
 }