Skip to content

[JENKINS-60902] adds the dependency of the Apache Mina sshd 2.5.1#54

Merged
jglick merged 9 commits intojenkinsci:masterfrom
kuisathaverat:apache_mina_2_6_x
Jun 8, 2021
Merged

[JENKINS-60902] adds the dependency of the Apache Mina sshd 2.5.1#54
jglick merged 9 commits intojenkinsci:masterfrom
kuisathaverat:apache_mina_2_6_x

Conversation

@kuisathaverat
Copy link
Contributor

@kuisathaverat kuisathaverat commented May 2, 2021

see JENKINS-60902

It adds the dependency of the Apache Mina sshd
It bumps the Apache Mina sshd library to 2.5.1
it bumps core to latest LTS 2.289.1

it replaces jenkinsci/sshd-plugin#33
it depends on JENKINS-55582 jenkinsci/sshd-plugin#46

@jglick jglick marked this pull request as draft May 3, 2021 15:11
…ase.java

Co-authored-by: Jesse Glick <jglick@cloudbees.com>
@kuisathaverat
Copy link
Contributor Author

this plugin does not need the dependency of the sshd-plugin, the Apache Mina SSHD library is only used on tests to create an SSH mock server, so I have replaced the sshd-plugin dependency with a test dependency of the Apache Mina SSHD library on version 2.5.1

Copy link
Member

@jglick jglick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIUC this only affects tests? So I can merge but there is no rush on a release?

@kuisathaverat
Copy link
Contributor Author

no, for this one there is no rush since the dependency is on tests

@kuisathaverat kuisathaverat changed the title [JENKINS-60902]Upgrade sshd-module to use SSHD 2.6.0 [JENKINS-60902]Upgrade sshd-module to use SSHD 2.5.1 Jun 7, 2021
@kuisathaverat kuisathaverat changed the title [JENKINS-60902]Upgrade sshd-module to use SSHD 2.5.1 [JENKINS-60902] adds the dependency of the Apache Mina sshd 2.5.1 Jun 7, 2021
@jglick jglick merged commit 4dd1ba7 into jenkinsci:master Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants