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 2ca95ee0b Demote JDK transport (#1599)
2ca95ee0b is described below

commit 2ca95ee0b88fe57c216fa7b3edaad555b80de51e
Author: Tamas Cservenak <ta...@cservenak.net>
AuthorDate: Mon Sep 22 16:13:20 2025 +0200

    Demote JDK transport (#1599)
    
    Originally we have this order of transports w/ priorities (present/absent 
in maven core):
    * wagon -1.0 (present)
    * apache 5.0 (present)
    * jdk 10.0 (present) ("default" transport)
    * jetty 15.0 (absent) (if user added it via extension or lib/ext it "wins")
    
    But lately we see that JDK one advantage of HTTP/2 support (that is already 
disabled due various issues) is shaded due several disadvantages (no support 
for HTTPS proxies, unstable, and unreliable)
    
    Hence, I propose to "demote" it (maybe even remove it from Maven core):
    * wagon -1.0 (present)
    * jdk 5.0 (present; and maybe remove it from maven core)
    * apache 10.0 (present; new "default" transport)
    * jetty 15.0 (absent)
    
    This makes Maven 4 not have "modern" HTTP transport by default, but apache 
transport is well battle tested and robust, with one issue: lack of HTTP/2 
support (that does improve download speeds).
    
    If we decide for this, it will have impact on Maven ITs as well that asser 
default transport.
---
 .../org/eclipse/aether/transport/apache/ApacheTransporterFactory.java   | 2 +-
 .../java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java    | 2 +-
 .../java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java    | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/maven-resolver-transport-apache/src/main/java/org/eclipse/aether/transport/apache/ApacheTransporterFactory.java
 
b/maven-resolver-transport-apache/src/main/java/org/eclipse/aether/transport/apache/ApacheTransporterFactory.java
index 3a3424a15..ec7412857 100644
--- 
a/maven-resolver-transport-apache/src/main/java/org/eclipse/aether/transport/apache/ApacheTransporterFactory.java
+++ 
b/maven-resolver-transport-apache/src/main/java/org/eclipse/aether/transport/apache/ApacheTransporterFactory.java
@@ -39,7 +39,7 @@ import static java.util.Objects.requireNonNull;
 public final class ApacheTransporterFactory implements HttpTransporterFactory {
     public static final String NAME = "apache";
 
-    private float priority = 5.0f;
+    private float priority = 10.0f;
 
     private final ChecksumExtractor checksumExtractor;
 
diff --git 
a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
 
b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
index bfc3c2098..b5c091992 100644
--- 
a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
+++ 
b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-11/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
@@ -40,7 +40,7 @@ import static java.util.Objects.requireNonNull;
 public final class JdkTransporterFactory implements HttpTransporterFactory {
     public static final String NAME = "jdk";
 
-    private float priority = 10.0f;
+    private float priority = 5.0f;
 
     private final ChecksumExtractor checksumExtractor;
 
diff --git 
a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-8/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
 
b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-8/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
index dcea7ce8b..a0792adb2 100644
--- 
a/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-8/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
+++ 
b/maven-resolver-transport-jdk-parent/maven-resolver-transport-jdk-8/src/main/java/org/eclipse/aether/transport/jdk/JdkTransporterFactory.java
@@ -44,7 +44,7 @@ public final class JdkTransporterFactory implements 
HttpTransporterFactory {
 
     private static final Logger LOGGER = 
LoggerFactory.getLogger(JdkTransporterFactory.class);
 
-    private float priority = 10.0f;
+    private float priority = 5.0f;
 
     @Inject
     public JdkTransporterFactory(ChecksumExtractor checksumExtractor, 
PathProcessor pathProcessor) {

Reply via email to