Skip to content

Commit 092eed6

Browse files
committed
Use SSHD 2.16.0
This removes the need for shipping a custom GitPackCommand as apache/mina-sshd#793 is fixed there. Force JGit 5 to use newer SSHD too.
1 parent 041492e commit 092eed6

File tree

7 files changed

+21
-186
lines changed

7 files changed

+21
-186
lines changed

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gittest/pom.xml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,6 @@
3131
<name>Maven SCM Git Provider TCK Tests</name>
3232
<description>Tests library for SCM Git Provider.</description>
3333

34-
<properties>
35-
<!-- newer versions clash with Sshd dependencies from JGit 5.x-->
36-
<minaSshdVersion>2.11.0</minaSshdVersion>
37-
</properties>
38-
3934
<dependencies>
4035
<dependency>
4136
<groupId>org.apache.maven.scm</groupId>

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gittest/src/main/java/org/apache/maven/scm/provider/git/GitSshServer.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,11 @@
3131
import java.util.ArrayList;
3232
import java.util.List;
3333

34-
import org.apache.maven.scm.provider.git.sshd.git.pack.GitPackCommandFactory;
3534
import org.apache.sshd.common.config.keys.KeyUtils;
3635
import org.apache.sshd.common.config.keys.writer.openssh.OpenSSHKeyEncryptionContext;
3736
import org.apache.sshd.common.config.keys.writer.openssh.OpenSSHKeyPairResourceWriter;
3837
import org.apache.sshd.git.GitLocationResolver;
38+
import org.apache.sshd.git.pack.GitPackCommandFactory;
3939
import org.apache.sshd.server.SshServer;
4040
import org.apache.sshd.server.auth.pubkey.KeySetPublickeyAuthenticator;
4141
import org.apache.sshd.server.auth.pubkey.PublickeyAuthenticator;
@@ -132,7 +132,6 @@ public Path resolveRootDirectory(String command, String[] args, ServerSession se
132132
return repositoryRoot;
133133
}
134134
};
135-
// use patched version of GitPackCommandFactory including https://github.com/apache/mina-sshd/pull/794
136135
sshServer.setCommandFactory(new GitPackCommandFactory(gitLocationResolver));
137136
sshServer.start();
138137
}

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gittest/src/main/java/org/apache/maven/scm/provider/git/sshd/git/pack/GitPackCommand.java

Lines changed: 0 additions & 109 deletions
This file was deleted.

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gittest/src/main/java/org/apache/maven/scm/provider/git/sshd/git/pack/GitPackCommandFactory.java

Lines changed: 0 additions & 65 deletions
This file was deleted.

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-gittest/src/main/java/org/apache/maven/scm/provider/git/sshd/git/pack/README.txt

Lines changed: 0 additions & 5 deletions
This file was deleted.

maven-scm-providers/maven-scm-providers-git/maven-scm-provider-jgit/pom.xml

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,22 @@
3535
<!-- version 6+ requires Java 11 -->
3636
<jgitVersion>5.13.3.202401111512-r</jgitVersion>
3737
</properties>
38+
39+
<dependencyManagement>
40+
<dependencies>
41+
<!-- upgrade SSHD used in JGit to newer version for being compatible with ITs -->
42+
<dependency>
43+
<groupId>org.apache.sshd</groupId>
44+
<artifactId>sshd-sftp</artifactId>
45+
<version>${minaSshdVersion}</version>
46+
</dependency>
47+
<dependency>
48+
<groupId>org.apache.sshd</groupId>
49+
<artifactId>sshd-osgi</artifactId>
50+
<version>${minaSshdVersion}</version>
51+
</dependency>
52+
</dependencies>
53+
</dependencyManagement>
3854
<dependencies>
3955
<dependency>
4056
<groupId>javax.inject</groupId>

maven-scm-providers/maven-scm-providers-git/pom.xml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,10 @@
3939
<module>maven-scm-provider-jgit</module>
4040
</modules>
4141

42+
<properties>
43+
<minaSshdVersion>2.16.0</minaSshdVersion>
44+
</properties>
45+
4246
<profiles>
4347
<profile>
4448
<id>tck-git</id>

0 commit comments

Comments
 (0)