net-ssh 7.2.0 → 8.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- checksums.yaml.gz.sig +0 -0
- data/.dockerignore +3 -0
- data/.github/pull_request_template.md +1 -0
- data/.github/workflows/changelog.yml +43 -0
- data/.github/workflows/ci-with-docker.yml +20 -9
- data/.github/workflows/ci.yml +11 -20
- data/.github/workflows/rubocop.yml +3 -1
- data/.gitignore +2 -0
- data/.rubocop.yml +1 -1
- data/.rubocop_todo.yml +4 -1
- data/CHANGES.txt +114 -0
- data/CONTRIBUTING.md +35 -0
- data/Dockerfile +7 -2
- data/Dockerfile.openssl3 +1 -1
- data/Dockerfile.zlib_ng +29 -0
- data/README.md +76 -57
- data/Rakefile +117 -30
- data/appveyor.yml +4 -16
- data/changelog.d/README.md +1 -0
- data/docker-compose.yml +2 -8
- data/lib/net/ssh/authentication/agent.rb +35 -11
- data/lib/net/ssh/authentication/ed25519.rb +141 -40
- data/lib/net/ssh/authentication/ed25519_loader.rb +14 -9
- data/lib/net/ssh/authentication/key_manager.rb +61 -18
- data/lib/net/ssh/authentication/methods/abstract.rb +2 -8
- data/lib/net/ssh/authentication/methods/publickey.rb +3 -1
- data/lib/net/ssh/authentication/pub_key_fingerprint.rb +1 -1
- data/lib/net/ssh/authentication/security_key.rb +71 -0
- data/lib/net/ssh/authentication/session.rb +34 -15
- data/lib/net/ssh/buffer.rb +6 -1
- data/lib/net/ssh/buffered_io.rb +41 -4
- data/lib/net/ssh/config.rb +30 -13
- data/lib/net/ssh/connection/channel.rb +49 -2
- data/lib/net/ssh/connection/session.rb +58 -8
- data/lib/net/ssh/errors.rb +24 -6
- data/lib/net/ssh/key_factory.rb +11 -4
- data/lib/net/ssh/known_hosts.rb +64 -11
- data/lib/net/ssh/multiplex/client.rb +162 -0
- data/lib/net/ssh/multiplex/control_path.rb +120 -0
- data/lib/net/ssh/multiplex/proxy_transport.rb +218 -0
- data/lib/net/ssh/multiplex.rb +130 -0
- data/lib/net/ssh/prompt.rb +3 -3
- data/lib/net/ssh/proxy/command.rb +28 -3
- data/lib/net/ssh/proxy/jump.rb +52 -4
- data/lib/net/ssh/service/forward.rb +38 -27
- data/lib/net/ssh/test/extensions.rb +3 -2
- data/lib/net/ssh/test/socket.rb +1 -1
- data/lib/net/ssh/test.rb +2 -1
- data/lib/net/ssh/transport/aes128_gcm.rb +40 -0
- data/lib/net/ssh/transport/aes256_gcm.rb +40 -0
- data/lib/net/ssh/transport/algorithms.rb +96 -34
- data/lib/net/ssh/transport/chacha20_poly1305_cipher.rb +89 -16
- data/lib/net/ssh/transport/chacha20_poly1305_cipher_loader.rb +20 -1
- data/lib/net/ssh/transport/cipher_factory.rb +32 -8
- data/lib/net/ssh/transport/constants.rb +1 -0
- data/lib/net/ssh/transport/gcm_cipher.rb +228 -0
- data/lib/net/ssh/transport/hmac/abstract.rb +16 -0
- data/lib/net/ssh/transport/hmac/sha1_etm.rb +12 -0
- data/lib/net/ssh/transport/hmac.rb +2 -0
- data/lib/net/ssh/transport/kex/curve25519_sha256.rb +41 -9
- data/lib/net/ssh/transport/kex/curve25519_sha256_loader.rb +14 -6
- data/lib/net/ssh/transport/kex/diffie_hellman_group1_sha1.rb +8 -0
- data/lib/net/ssh/transport/kex/diffie_hellman_group_exchange_sha1.rb +45 -15
- data/lib/net/ssh/transport/openssl.rb +2 -1
- data/lib/net/ssh/transport/packet_stream.rb +19 -3
- data/lib/net/ssh/transport/server_version.rb +33 -13
- data/lib/net/ssh/transport/session.rb +60 -46
- data/lib/net/ssh/transport/state.rb +1 -1
- data/lib/net/ssh/verifiers/always.rb +43 -8
- data/lib/net/ssh/version.rb +2 -2
- data/lib/net/ssh.rb +142 -62
- data/net-ssh-public_cert.pem +19 -18
- data/net-ssh.gemspec +7 -8
- data/rakelib/changelog.rb +229 -0
- data.tar.gz.sig +0 -0
- metadata +86 -53
- metadata.gz.sig +1 -1
- data/Gemfile.noed25519 +0 -12
- data/Gemfile.norbnacl +0 -12
- data/lib/net/ssh/version.rb.old +0 -68
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: fc22933c7ce50930b6118479820acb8d978d855baade0499b94065d7e245c016
|
|
4
|
+
data.tar.gz: 3340e9bd30ac3781da55ffdc0362626e32ad8b0c2db0b2352bf879dc6fe743bc
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: 226a157e20beb49c008e4a3eb3e15a93ea23b7d0b2935f3c0fa4111d4e43815f6a77b0ef2f22e2bab7123f9410f99655459dc42b94c30ce63a295be77dc3b2ce
|
|
7
|
+
data.tar.gz: fb9269c31afc40374ef1b30cc58008d916cf41f3a633596e2fb04f924917f33e966c3b556df56f3cb10fbc24190d956ac9b59b2d57cb7b23204bded86f5316c6
|
checksums.yaml.gz.sig
CHANGED
|
Binary file
|
data/.dockerignore
CHANGED
|
@@ -0,0 +1 @@
|
|
|
1
|
+
<!-- After opening the PR, add a changelog.d/<PR number>.<type>.txt fragment (none, change, security, breaking or breaking-security) instead of editing CHANGES.txt; see CONTRIBUTING.md#changelog. -->
|
|
@@ -0,0 +1,43 @@
|
|
|
1
|
+
name: Changelog
|
|
2
|
+
|
|
3
|
+
on:
|
|
4
|
+
pull_request:
|
|
5
|
+
types: [opened, synchronize, reopened, labeled, unlabeled]
|
|
6
|
+
|
|
7
|
+
permissions:
|
|
8
|
+
contents: read
|
|
9
|
+
|
|
10
|
+
jobs:
|
|
11
|
+
changelog:
|
|
12
|
+
if: ${{ github.event.pull_request.user.login != 'dependabot[bot]' }}
|
|
13
|
+
runs-on: ubuntu-latest
|
|
14
|
+
steps:
|
|
15
|
+
- uses: actions/checkout@v4
|
|
16
|
+
with:
|
|
17
|
+
# HEAD is the PR merged into the base branch; HEAD^1 is the base it was merged into.
|
|
18
|
+
fetch-depth: 2
|
|
19
|
+
- name: Set up Ruby 3.1
|
|
20
|
+
uses: ruby/setup-ruby@v1
|
|
21
|
+
with:
|
|
22
|
+
ruby-version: 3.1
|
|
23
|
+
- name: Validate fragments
|
|
24
|
+
run: rake changelog:check
|
|
25
|
+
- name: Require a fragment
|
|
26
|
+
if: ${{ !contains(github.event.pull_request.labels.*.name, 'skip-changelog') }}
|
|
27
|
+
env:
|
|
28
|
+
PR: ${{ github.event.number }}
|
|
29
|
+
run: |
|
|
30
|
+
changed=$(git diff --name-only HEAD^1 HEAD)
|
|
31
|
+
if echo "$changed" | grep -x 'CHANGES\.txt' > /dev/null; then
|
|
32
|
+
echo "::error::Don't edit CHANGES.txt, add a changelog.d/$PR.<type>.txt fragment instead, see CONTRIBUTING.md#changelog"
|
|
33
|
+
exit 1
|
|
34
|
+
fi
|
|
35
|
+
own=$(git diff --name-only --diff-filter=AMR HEAD^1 HEAD -- changelog.d/ | grep -E "^changelog\.d/$PR\.[a-z-]+(\.[0-9]+)?\.txt$" || true)
|
|
36
|
+
if [ -n "$own" ]; then
|
|
37
|
+
echo "Changelog fragments:"; echo "$own"
|
|
38
|
+
elif ! echo "$changed" | grep -v '^changelog\.d/' > /dev/null; then
|
|
39
|
+
echo "Only changelog.d/ changed, no fragment needed"
|
|
40
|
+
else
|
|
41
|
+
echo "::error::Add a changelog fragment named changelog.d/$PR.<type>.txt (use $PR.none.txt if nothing user-visible changed), see CONTRIBUTING.md#changelog"
|
|
42
|
+
exit 1
|
|
43
|
+
fi
|
|
@@ -12,22 +12,19 @@ jobs:
|
|
|
12
12
|
|
|
13
13
|
steps:
|
|
14
14
|
- name: Checkout code
|
|
15
|
-
uses: actions/checkout@
|
|
15
|
+
uses: actions/checkout@v4
|
|
16
16
|
|
|
17
17
|
- name: Build docker images
|
|
18
|
-
run: docker
|
|
19
|
-
|
|
20
|
-
- name: Run ruby 2.6
|
|
21
|
-
run: docker-compose run ruby-2.6
|
|
18
|
+
run: docker compose build
|
|
22
19
|
|
|
23
20
|
- name: Run ruby 2.7
|
|
24
|
-
run: docker
|
|
21
|
+
run: docker compose run ruby-2.7
|
|
25
22
|
|
|
26
23
|
- name: Run ruby 3.0
|
|
27
|
-
run: docker
|
|
24
|
+
run: docker compose run ruby-3.0
|
|
28
25
|
|
|
29
26
|
- name: Run ruby 3.1
|
|
30
|
-
run: docker
|
|
27
|
+
run: docker compose run ruby-3.1
|
|
31
28
|
|
|
32
29
|
test_openssl3:
|
|
33
30
|
name: Run test suite with docker and openssl 3.0
|
|
@@ -35,10 +32,24 @@ jobs:
|
|
|
35
32
|
|
|
36
33
|
steps:
|
|
37
34
|
- name: Checkout code
|
|
38
|
-
uses: actions/checkout@
|
|
35
|
+
uses: actions/checkout@v4
|
|
39
36
|
|
|
40
37
|
- name: Build docker images
|
|
41
38
|
run: docker build -t netssh_openssl3 -f Dockerfile.openssl3 .
|
|
42
39
|
|
|
43
40
|
- name: Run ruby 3.0 with openssl3
|
|
44
41
|
run: docker run --rm netssh_openssl3
|
|
42
|
+
|
|
43
|
+
test_zlib_ng:
|
|
44
|
+
name: Run test suite with docker and zlib-ng
|
|
45
|
+
runs-on: ubuntu-latest
|
|
46
|
+
|
|
47
|
+
steps:
|
|
48
|
+
- name: Checkout code
|
|
49
|
+
uses: actions/checkout@v4
|
|
50
|
+
|
|
51
|
+
- name: Build docker images
|
|
52
|
+
run: docker build -t netssh_zlib_ng -f Dockerfile.zlib_ng .
|
|
53
|
+
|
|
54
|
+
- name: Run ruby with zlib-ng
|
|
55
|
+
run: docker run --rm netssh_zlib_ng
|
data/.github/workflows/ci.yml
CHANGED
|
@@ -7,7 +7,7 @@ jobs:
|
|
|
7
7
|
runs-on: ubuntu-22.04
|
|
8
8
|
strategy:
|
|
9
9
|
matrix:
|
|
10
|
-
ruby-version: [2.
|
|
10
|
+
ruby-version: [2.7.8, 3.0.6, 3.1.3, 3.2.1, 3.3, 3.4, '4.0']
|
|
11
11
|
steps:
|
|
12
12
|
- uses: actions/checkout@v3
|
|
13
13
|
|
|
@@ -21,16 +21,16 @@ jobs:
|
|
|
21
21
|
python-version: 3.8
|
|
22
22
|
|
|
23
23
|
- name: Cache bundler
|
|
24
|
-
uses: actions/cache@
|
|
24
|
+
uses: actions/cache@v4
|
|
25
25
|
id: bundler-cache
|
|
26
26
|
with:
|
|
27
27
|
path: vendor/bundle
|
|
28
28
|
key: ${{ runner.os }}-${{ matrix.ruby-version }}-gem-v3-${{ hashFiles('**/Gemfile') }}-${{ hashFiles('**/net-ssh.gemspec') }}
|
|
29
29
|
restore-keys: |
|
|
30
30
|
${{ runner.os }}-${{ matrix.ruby-version }}-gem-v3-
|
|
31
|
-
|
|
31
|
+
|
|
32
32
|
- name: Cache pip
|
|
33
|
-
uses: actions/cache@
|
|
33
|
+
uses: actions/cache@v4
|
|
34
34
|
id: pip-cache
|
|
35
35
|
with:
|
|
36
36
|
path: ~/.cache/pip
|
|
@@ -39,14 +39,14 @@ jobs:
|
|
|
39
39
|
${{ runner.os }}-pip-v1
|
|
40
40
|
- name: Bundle install
|
|
41
41
|
run: |
|
|
42
|
-
gem install bundler
|
|
42
|
+
gem install bundler ${{ startsWith(matrix.ruby-version, '2.7.') && '-v 2.4.22' || '' }}
|
|
43
43
|
bundle config set path 'vendor/bundle'
|
|
44
44
|
bundle config set --local path 'vendor/bundle'
|
|
45
|
-
bundle install --jobs 4 --retry 3
|
|
46
|
-
BUNDLE_GEMFILE=./Gemfile.noed25519 bundle install --jobs 4 --retry 3 --path vendor/bundle
|
|
45
|
+
bundle install --jobs 4 --retry 3
|
|
47
46
|
env:
|
|
48
47
|
BUNDLE_PATH: vendor/bundle
|
|
49
|
-
|
|
48
|
+
|
|
49
|
+
|
|
50
50
|
- name: Add to etc/hosts
|
|
51
51
|
run: |
|
|
52
52
|
sudo echo "127.0.0.1 gateway.netssh" | sudo tee -a /etc/hosts
|
|
@@ -77,17 +77,8 @@ jobs:
|
|
|
77
77
|
env:
|
|
78
78
|
NET_SSH_RUN_INTEGRATION_TESTS: 1
|
|
79
79
|
CI: 1
|
|
80
|
-
|
|
81
|
-
run: bundle exec rake test
|
|
82
|
-
env:
|
|
83
|
-
BUNDLE_GEMFILE: ./Gemfile.norbnacl
|
|
84
|
-
NET_SSH_RUN_INTEGRATION_TESTS: 1
|
|
85
|
-
CI: 1
|
|
86
|
-
- name: Run Tests (without ed25519)
|
|
87
|
-
run: bundle exec rake test
|
|
88
|
-
env:
|
|
89
|
-
BUNDLE_GEMFILE: ./Gemfile.noed25519
|
|
90
|
-
NET_SSH_RUN_INTEGRATION_TESTS: 1
|
|
91
|
-
CI: 1
|
|
80
|
+
RUBYOPT: '--enable=frozen-string-literal'
|
|
92
81
|
- name: Run test helper test
|
|
93
82
|
run: bundle exec rake test_test
|
|
83
|
+
env:
|
|
84
|
+
RUBYOPT: '--enable=frozen-string-literal'
|
data/.gitignore
CHANGED
data/.rubocop.yml
CHANGED
data/.rubocop_todo.yml
CHANGED
|
@@ -235,7 +235,7 @@ Lint/UselessTimes:
|
|
|
235
235
|
# Offense count: 205
|
|
236
236
|
# Configuration parameters: IgnoredMethods, CountRepeatedAttributes.
|
|
237
237
|
Metrics/AbcSize:
|
|
238
|
-
Max:
|
|
238
|
+
Max: 76
|
|
239
239
|
|
|
240
240
|
# Offense count: 16
|
|
241
241
|
# Configuration parameters: CountComments, CountAsOne, ExcludedMethods, IgnoredMethods.
|
|
@@ -335,6 +335,7 @@ Naming/MemoizedInstanceVariableName:
|
|
|
335
335
|
Naming/MethodName:
|
|
336
336
|
Exclude:
|
|
337
337
|
- 'lib/net/ssh/authentication/ed25519_loader.rb'
|
|
338
|
+
- 'lib/net/ssh/transport/chacha20_poly1305_cipher_loader.rb'
|
|
338
339
|
- 'lib/net/ssh/transport/kex/curve25519_sha256_loader.rb'
|
|
339
340
|
- 'test/authentication/test_agent.rb'
|
|
340
341
|
- 'test/authentication/test_session.rb'
|
|
@@ -758,10 +759,12 @@ Style/OpenStructUse:
|
|
|
758
759
|
Exclude:
|
|
759
760
|
- 'test/authentication/test_ed25519.rb'
|
|
760
761
|
- 'test/common.rb'
|
|
762
|
+
- 'test/test_known_hosts.rb'
|
|
761
763
|
- 'test/transport/kex/test_curve25519_sha256.rb'
|
|
762
764
|
- 'test/transport/kex/test_diffie_hellman_group1_sha1.rb'
|
|
763
765
|
- 'test/transport/kex/test_ecdh_sha2_nistp256.rb'
|
|
764
766
|
- 'test/verifiers/test_always.rb'
|
|
767
|
+
- 'test/verifiers/test_cert_authority_checks.rb'
|
|
765
768
|
|
|
766
769
|
# Offense count: 16
|
|
767
770
|
# Configuration parameters: AllowedMethods.
|
data/CHANGES.txt
CHANGED
|
@@ -1,3 +1,117 @@
|
|
|
1
|
+
=== 8.0.0
|
|
2
|
+
|
|
3
|
+
* No changes since 8.0.0 rc5.
|
|
4
|
+
|
|
5
|
+
=== 8.0.0 rc5
|
|
6
|
+
|
|
7
|
+
* BREAKING: Security: a custom `verify_host_key` verifier must now implement `verify_signature` (deprecated since 5.1.0) or ArgumentError is raised; such verifiers were previously wrapped so the server's host key signature was never checked [#1050]
|
|
8
|
+
* BREAKING: Unsupported key types now raise `Net::SSH::UnsupportedKeyType` (a `Net::SSH::Exception`) instead of `NotImplementedError`; code that rescued `NotImplementedError` around key loading or connecting should rescue the new class [#945, #1060]
|
|
9
|
+
* BREAKING: `Net::SSH.start` raises ArgumentError for option values that are `nil` or arrays containing `nil` (deprecated since 4.0.0); omit the option to use the default or ssh_config value, the error names a value that turns it off where one exists (e.g. `proxy: false`, `keys: []`, `config: false`). `password: nil` and `passphrase: nil` are still accepted [#458, #1053]
|
|
10
|
+
* BREAKING: removed the `:paranoid` option (deprecated since 4.2.0); use `:verify_host_key` with the same values. Passing `:paranoid` raises ArgumentError [#1050]
|
|
11
|
+
* BREAKING: removed the `verify_host_key` values `false`, `true`, `:very` and `:secure` (deprecated since 5.0.0); use `:never`, `:accept_new_or_local_tunnel`, `:accept_new` and `:always` respectively. The old values raise ArgumentError naming the replacement [#1050]
|
|
12
|
+
* BREAKING: `Net::SSH::HostKeyEntries::PubKey` (the known_hosts entries returned by `KnownHosts.search_for`) no longer forwards methods to the wrapped key (deprecated since 7.0.0); use `ssh_type`, `to_blob` or `matches_key?`, or the new `key` reader for the wrapped public key [#1050]
|
|
13
|
+
* Security: `append_all_supported_algorithms: true` no longer enables the `none` cipher and MAC; name them explicitly (`encryption: '+none'`, `hmac: '+none'`) to use them [#1051]
|
|
14
|
+
* Security: `diffie-hellman-group-exchange-sha1/sha256` now requires groups of at least 2048 bits and requests sizes from the negotiated ciphers, as OpenSSH since 7.2, and rejects invalid server groups and DH public values; like OpenSSH, the requested group size is capped at 4096 bits for Cisco servers. Pass `minimum_dh_bits: 1024` to connect to legacy servers that only offer 1024-bit groups. `minimum_dh_bits` values other than 1024 no longer fail key exchange with "could not verify server signature" [#1049]
|
|
15
|
+
* Security: `diffie-hellman-group14-sha1` is no longer offered by default for key exchange, as in OpenSSH since 8.2; add `KexAlgorithms +diffie-hellman-group14-sha1` in ssh_config (or `kex: '+diffie-hellman-group14-sha1'`) or pass `append_all_supported_algorithms: true` to re-enable [#705, #1042]
|
|
16
|
+
* Changelog entries are now added as `changelog.d/<PR number>.<type>.txt` fragment files instead of editing CHANGES.txt, and CI requires one per PR; see CONTRIBUTING.md [#1058]
|
|
17
|
+
* `Net::SSH.start(host, user, control_master: :auto)` reuses an already running OpenSSH ControlMaster found via `:control_path` or ssh_config `ControlPath`, falling back to a normal connection when there is none [#443, #1057]
|
|
18
|
+
* Default `max_win_size` is now 2 MB as in OpenSSH (was 128 KB), which avoids truncated output from some devices (e.g. Mikrotik); pass `max_win_size: 0x20000` for the old behaviour [#726, #1055]
|
|
19
|
+
* Added `hmac-sha1-etm@openssh.com`, offered by default after the SHA2 etm MACs as in OpenSSH [#985, #1056]
|
|
20
|
+
* `Net::SSH::Proxy::Jump` `options:` values may be an Array for ssh_config options that can be given more than once (`IdentityFile`, `CertificateFile`, `SendEnv`); each element is passed as its own `-o Key=value`, an empty Array passes nothing, and a `nil` value (alone or in an Array) raises ArgumentError as in `Net::SSH.start` [#1052]
|
|
21
|
+
* `ssh-rsa-cert-v00@openssh.com` host keys are deprecated and warn when named in `host_key`/`HostKeyAlgorithms` or negotiated; net-ssh cannot parse v00 certificates, use `ssh-rsa-cert-v01@openssh.com` or a modern key type [#1051]
|
|
22
|
+
* README: weak algorithms (CBC/3DES/Blowfish ciphers, `ssh-dss`, `diffie-hellman-group1-sha1`, old MACs, ...) stay available as opt-ins for legacy servers rather than being removed in 8.0 [#1051]
|
|
23
|
+
* An ssh-agent that fails while listing keys (e.g. closes the connection) is now skipped and key files are still tried, as in OpenSSH, instead of aborting authentication [#867, #1048]
|
|
24
|
+
* `%r` in a ProxyCommand now defaults to the connection's user, as in OpenSSH, instead of raising unless `:remote_user` is set [#881, #787, #1047]
|
|
25
|
+
* `Net::SSH.start` no longer modifies the options hash passed to it [#989, #1046]
|
|
26
|
+
* `Session#exec!` with a block now returns an empty string with `exitstatus` (like the non-block form) instead of `nil` [#848, #1045]
|
|
27
|
+
* ssh_config `KexAlgorithms` is now honoured and maps to `:kex`, including `+`, `-` and `^`; it was previously ignored [#1042]
|
|
28
|
+
* `Net::SSH::Proxy::Jump.new` accepts `options:`, a Hash of ssh_config options passed to the jump host's ssh command as escaped `-o Key=value` arguments, e.g. `options: { 'StrictHostKeyChecking' => 'no' }` [#915, #1041]
|
|
29
|
+
* `Net::SSH::Proxy::Jump` no longer fails with "unknown key" or substitutes the target host when the `config:` path contains `%` [#1041]
|
|
30
|
+
|
|
31
|
+
=== 8.0.0 rc4
|
|
32
|
+
|
|
33
|
+
* Security: RSA signatures using SHA-1 (`ssh-rsa`, `ssh-rsa-cert-v01@openssh.com`) are no longer offered by default for host keys or public key authentication, as in OpenSSH since 8.8; RSA keys keep working via rsa-sha2-256/512. For servers that only support SHA-1 add `HostKeyAlgorithms +ssh-rsa` / `PubkeyAcceptedAlgorithms +ssh-rsa` in ssh_config (or `host_key: '+ssh-rsa'` / `pubkey_algorithms: %w[+ssh-rsa]`), or pass `append_all_supported_algorithms: true` [#1039]
|
|
34
|
+
* Security: ssh_config `PubkeyAcceptedAlgorithms` (and its old name `PubkeyAcceptedKeyTypes`) is now honoured, including `+`, `-` and `^`, and `pubkey_algorithms` now applies to all key types instead of only RSA, adding `rsa-sha2-512`; DSA (`ssh-dss`) keys are no longer offered by default, as in OpenSSH since 7.0 — add `+ssh-dss` or pass `append_all_supported_algorithms: true` to re-enable [#923]
|
|
35
|
+
* Security: ssh_config `Ciphers`, `MACs`, `KexAlgorithms` and `HostKeyAlgorithms` lists starting with `+`, `-` or `^` now append to, remove from or prepend to the default list, as in OpenSSH, instead of silently enabling every supported algorithm including weak ones that are off by default; every listed item is honoured and `-` accepts wildcards. Pass `append_all_supported_algorithms: true` to also get the full supported list [#925]
|
|
36
|
+
* Support agent-backed FIDO/U2F security keys (sk-ecdsa-sha2-nistp256@openssh.com, sk-ssh-ed25519@openssh.com) for publickey auth [#1007]
|
|
37
|
+
* Port forwarding no longer buffers a whole stream in memory when one side reads slowly: like OpenSSH, window is granted only as data is written to the local socket, and the local socket is read only while the remote window has room. An application that writes everything to a forwarded port before reading anything now blocks once the window fills, as it would with `ssh -L` or a direct TCP connection; the old unbounded buffering hid this [#1037, #1003]
|
|
38
|
+
* Data forwarded to a local socket just before the remote end sends EOF or closes the channel is no longer truncated or dropped: it is written out before the socket is shut down or closed, without blocking the event loop until the local application reads [#1037]
|
|
39
|
+
* Multi-factor authentication works when the server's AuthenticationMethods order differs from net-ssh's: after each partial success the allowed methods are tried again, as OpenSSH does [#873, #872]
|
|
40
|
+
* `HostKeyError#host`, `#port` and `#ip` now return the connected host, port and IP (`ip` is nil through a ProxyCommand) instead of always nil [#963]
|
|
41
|
+
* OpenSSH-format private keys with Windows (CRLF) line endings now load [#922, #921]
|
|
42
|
+
* Send CHANNEL_EOF before CHANNEL_CLOSE when closing a session's remaining channels, as OpenSSH does; fixes hangs at close with servers that wait for EOF, such as some SFTP servers [#672]
|
|
43
|
+
* ssh_config: trailing `# comments` are now ignored, as in OpenSSH; a `#` inside quotes or in the middle of a word is kept [#990]
|
|
44
|
+
|
|
45
|
+
=== 8.0.0 rc3
|
|
46
|
+
|
|
47
|
+
* Fix "Calling Net::SSH::Buffer methods on HostKeyEntries PubKey is deprecated" warnings printed on every connection to a host in known_hosts [#1026, #1027]
|
|
48
|
+
* `ProxyJump none` now means no proxy instead of jumping through a host named "none", and `ProxyCommand none` is matched case-insensitively, as in OpenSSH [#893, #972, #1030]
|
|
49
|
+
* Send SIGHUP to the ProxyCommand process on close, as OpenSSH does, so tunnels such as EC2 Instance Connect Endpoint no longer hang at exit [#947]
|
|
50
|
+
* Reconnect when the ssh-agent closes the connection on the legacy version request (e.g. older Proton Pass) instead of raising FrozenError [#1008]
|
|
51
|
+
* ssh_config: parse `Key = value` lines like OpenSSH (the value no longer keeps the leading space, and the keyword ends at the first `=`), and honor ProxyJump/ProxyCommand before the first Host [#1034]
|
|
52
|
+
|
|
53
|
+
=== 8.0.0 rc2
|
|
54
|
+
|
|
55
|
+
* Security: refuse agent channels opened by the server unless agent forwarding was requested (`forward_agent`, ForwardAgent, or `Forward#agent`); any use of port forwarding let a malicious server use the local ssh-agent [GHSA-qpg5-fx6p-8cq3]
|
|
56
|
+
* Security: reject host and user names containing shell metacharacters, and non-numeric ports, before substituting them into ProxyCommand/ProxyJump commands, as OpenSSH does since 9.6 [GHSA-3x7f-q7wf-fp73]
|
|
57
|
+
* Security: reject OpenSSH private keys claiming more than 2**20 bcrypt rounds, and only run the KDF once a passphrase is given [GHSA-rpq3-v334-f5pm]
|
|
58
|
+
* Security: bound the server's identification banner like OpenSSH (8192-byte lines, 1024 lines), and apply `:timeout` to the whole version exchange instead of only its first byte [GHSA-r75w-7fhp-84w8]
|
|
59
|
+
* Security: reject packet lengths below 5 or above 256 KB, as OpenSSH does [GHSA-pfqh-p9vh-wx4p]
|
|
60
|
+
* Security: treat unexpected packets during the initial key exchange as a protocol error, and bound packets queued during a re-key [GHSA-ggxh-cfwq-5xqc]
|
|
61
|
+
|
|
62
|
+
=== 8.0.0 rc1
|
|
63
|
+
|
|
64
|
+
* BREAKING: TCP_NODELAY is now set on the connection socket by default, as OpenSSH does; pass `no_delay: false` to keep Nagle's algorithm on [#1017]
|
|
65
|
+
* Ruby 4.0 support: add `logger` as a runtime dependency; Windows Pageant users on Ruby 4.0 need `fiddle` in their Gemfile [#984, #1012]
|
|
66
|
+
* Close the socket when connection setup fails, e.g. on a server version timeout [#1013]
|
|
67
|
+
* Ignore SSH_MSG_EXT_INFO from servers that send it unrequested [#1020]
|
|
68
|
+
* Fix `undefined method 'to_pem'` for ECDSA key_data when an agent is running [#982]
|
|
69
|
+
* Agent forwarding connects to the agent set by `identity_agent`/IdentityAgent [#974]
|
|
70
|
+
* Keyboard-interactive prompts show input when the server asks for it to be echoed [#1014]
|
|
71
|
+
* Raise Net::SSH::Exception on a truncated KEXINIT [#1016]
|
|
72
|
+
* Remote forwarding no longer prints diagnostics to stdout [#953]
|
|
73
|
+
|
|
74
|
+
=== 8.0.0 beta3
|
|
75
|
+
|
|
76
|
+
* Verify @cert-authority host certificate constraints — principals (shell-glob, matching OpenSSH), validity window, host type, and critical options — instead of trusting any CA-signed certificate for any host [#1009]
|
|
77
|
+
|
|
78
|
+
=== 8.0.0 beta1
|
|
79
|
+
|
|
80
|
+
* BREAKING: require Ruby 2.7 and the openssl gem >= 3.2 [#1001]
|
|
81
|
+
* Use OpenSSL for Ed25519, X25519 and ChaCha20-Poly1305; the ed25519, x25519 and rbnacl gems are no longer needed [#1001]
|
|
82
|
+
* Support AEAD-encrypted OpenSSH private keys [#1001]
|
|
83
|
+
|
|
84
|
+
=== 7.3.3
|
|
85
|
+
|
|
86
|
+
* Support .pub file as IdentityFile to filter/prioritize agent keys (1Password SSH agent use case) [#942]
|
|
87
|
+
* Fix cert signing regression when using a .pub companion file without an agent [#1006]
|
|
88
|
+
* Fix unreadable .pub IdentityFile silently skipped instead of raising [#1006]
|
|
89
|
+
* Allow multiple UserKnownHostsFile entries (space-separated, matching ssh_config(5)) [#979]
|
|
90
|
+
* Stop mutating ENV['HOME'] at require time [#997]
|
|
91
|
+
* Fix ED25519 loader with frozen string literals [#995]
|
|
92
|
+
|
|
93
|
+
=== 7.3.2 rc2
|
|
94
|
+
|
|
95
|
+
* Fix Ruby 2.6 compat: use public_to_der fallback for EC public keys [#993]
|
|
96
|
+
* Fix test framework hang: preserve StringIO position in idle!
|
|
97
|
+
|
|
98
|
+
=== 7.3.2 rc1
|
|
99
|
+
|
|
100
|
+
* Support frozen strings [#958]
|
|
101
|
+
|
|
102
|
+
=== 7.3.0 rc0
|
|
103
|
+
|
|
104
|
+
* aes(128|256)gcm [#946]
|
|
105
|
+
|
|
106
|
+
=== 7.2.2
|
|
107
|
+
|
|
108
|
+
* ruby 3.3.0: base64 fix
|
|
109
|
+
|
|
110
|
+
=== 7.2.1 rc1
|
|
111
|
+
|
|
112
|
+
* feat: allow load of certkey from string [#926]
|
|
113
|
+
* fix: fix for Socket#recv returning nil on ruby 3.3.0 [#928]
|
|
114
|
+
|
|
1
115
|
=== 7.2.0
|
|
2
116
|
|
|
3
117
|
* Add debugging information for algorithm of pubkey in use [#918]
|
data/CONTRIBUTING.md
ADDED
|
@@ -0,0 +1,35 @@
|
|
|
1
|
+
# Contributing to net-ssh
|
|
2
|
+
|
|
3
|
+
Run the unit tests with `bundle exec rake test` and the linter with `bundle exec rubocop`; see [README.md](README.md) and [DEVELOPMENT.md](DEVELOPMENT.md) for integration tests and debugging tips.
|
|
4
|
+
|
|
5
|
+
## Changelog
|
|
6
|
+
|
|
7
|
+
Don't edit `CHANGES.txt`. Instead, add a fragment file to `changelog.d/` in your PR; the release's version bump (`rake vbump:*`) moves all fragments into a new `=== <version>` section at the top of `CHANGES.txt` and deletes them.
|
|
8
|
+
|
|
9
|
+
A fragment is named `<number>.<type>.txt`, where `<number>` is the PR number (so open the PR first, then add the fragment), and contains a single UTF-8 paragraph: the entry as it should read in `CHANGES.txt`, without the bullet or prefix. Lines are joined, so it can be wrapped. If it doesn't end with a reference list such as `[#985, #1056]`, `[GHSA-xxxx-xxxx-xxxx]` or `[CVE-2024-12345, #1060]`, ` [#<number>]` is appended; write the list yourself to reference issues or advisories too.
|
|
10
|
+
|
|
11
|
+
| Type | Written as | Use for |
|
|
12
|
+
| ------------------- | ------------------------------ | ------------------------------------------------------------ |
|
|
13
|
+
| `breaking-security` | `* BREAKING: Security: <text>` | an incompatible change made for security |
|
|
14
|
+
| `breaking` | `* BREAKING: <text>` | an incompatible change |
|
|
15
|
+
| `security` | `* Security: <text>` | a security fix or hardening |
|
|
16
|
+
| `change` | `* <text>` | any other user-visible fix or feature |
|
|
17
|
+
| `none` | nothing | PRs with no user-visible change (tests, CI, docs, refactors) |
|
|
18
|
+
|
|
19
|
+
Examples:
|
|
20
|
+
|
|
21
|
+
```sh
|
|
22
|
+
rake "changelog:new[1058,change,Session#exec! now returns the exit status]"
|
|
23
|
+
# changelog.d/1058.change.txt -> * Session#exec! now returns the exit status [#1058]
|
|
24
|
+
|
|
25
|
+
echo 'Removed the `:paranoid` option; use `:verify_host_key` [#1050]' > changelog.d/1050.breaking.txt
|
|
26
|
+
echo 'Reject packet lengths above 256 KB, as OpenSSH does [GHSA-pfqh-p9vh-wx4p]' > changelog.d/1060.security.txt
|
|
27
|
+
echo 'Custom verifiers must implement `verify_signature` [#1050]' > changelog.d/1050.breaking-security.txt
|
|
28
|
+
touch changelog.d/1061.none.txt
|
|
29
|
+
```
|
|
30
|
+
|
|
31
|
+
For several entries from one PR add a suffix: `1050.breaking.txt`, `1050.breaking.2.txt` (`changelog:new` picks the next free one).
|
|
32
|
+
|
|
33
|
+
In a release section, entries are grouped as `BREAKING: Security:`, `BREAKING:`, `Security:`, then the rest; within a group fragments are ordered by number, newest first, followed by any hand-written entries from an `=== Unreleased` section in their written order. A final release with nothing to report still gets a `=== <version>` section saying `No changes since <previous version>.`
|
|
34
|
+
|
|
35
|
+
`rake changelog:check` validates the fragments and the `=== Unreleased` section, and `rake changelog:preview` prints the section the next release would add. CI fails a PR that edits `CHANGES.txt` or doesn't add or modify a `changelog.d/<PR number>.<type>.txt` fragment, unless it only touches `changelog.d/` or a maintainer adds the `skip-changelog` label; Dependabot PRs are skipped.
|
data/Dockerfile
CHANGED
|
@@ -1,6 +1,11 @@
|
|
|
1
1
|
ARG RUBY_VERSION=3.1
|
|
2
2
|
FROM ruby:${RUBY_VERSION}
|
|
3
3
|
|
|
4
|
+
ARG BUNDLERV=
|
|
5
|
+
|
|
6
|
+
# Ruby 2.7 and 3.0 images are built on Debian bullseye, which only archive.debian.org still serves.
|
|
7
|
+
RUN if grep -q bullseye /etc/os-release; then sed -i 's|deb.debian.org|archive.debian.org|' /etc/apt/sources.list; fi
|
|
8
|
+
|
|
4
9
|
RUN apt update && apt install -y openssh-server sudo netcat-openbsd \
|
|
5
10
|
&& useradd --create-home --shell '/bin/bash' --comment 'NetSSH' 'net_ssh_1' \
|
|
6
11
|
&& useradd --create-home --shell '/bin/bash' --comment 'NetSSH' 'net_ssh_2' \
|
|
@@ -20,8 +25,8 @@ COPY Gemfile net-ssh.gemspec $INSTALL_PATH/
|
|
|
20
25
|
|
|
21
26
|
COPY lib/net/ssh/version.rb $INSTALL_PATH/lib/net/ssh/version.rb
|
|
22
27
|
|
|
23
|
-
RUN gem install bundler && bundle install
|
|
28
|
+
RUN gem install bundler ${BUNDLERV} && bundle install
|
|
24
29
|
|
|
25
30
|
COPY . $INSTALL_PATH/
|
|
26
31
|
|
|
27
|
-
CMD service ssh start && rake test
|
|
32
|
+
CMD service ssh start && rake test
|
data/Dockerfile.openssl3
CHANGED
data/Dockerfile.zlib_ng
ADDED
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
FROM fedora:latest
|
|
2
|
+
|
|
3
|
+
ENV INSTALL_PATH="/netssh"
|
|
4
|
+
|
|
5
|
+
# Fedora's OpenSSL 3.x blocks SHA-1 for RSA/DSA signing by default.
|
|
6
|
+
# - LEGACY crypto policy relaxes the system-wide restrictions.
|
|
7
|
+
# - OPENSSL_CONF points to our config which activates the legacy provider,
|
|
8
|
+
# sets SECLEVEL=0, and sets rh-allow-sha1-signatures=yes (Fedora/RHEL
|
|
9
|
+
# OpenSSL patch that gates SHA-1 signing independently of the policy level).
|
|
10
|
+
# OPENSSL_CONF must be set as a Docker ENV so it is in the environment before
|
|
11
|
+
# Ruby/OpenSSL initialises — setting it in the Rakefile is too late.
|
|
12
|
+
ENV OPENSSL_CONF="/netssh/test/openssl3.conf"
|
|
13
|
+
|
|
14
|
+
RUN dnf install -y openssl openssl-devel ruby ruby-devel git make gcc redhat-rpm-config zlib-ng crypto-policies-scripts && \
|
|
15
|
+
update-crypto-policies --set LEGACY
|
|
16
|
+
|
|
17
|
+
WORKDIR $INSTALL_PATH
|
|
18
|
+
|
|
19
|
+
COPY Gemfile net-ssh.gemspec $INSTALL_PATH/
|
|
20
|
+
|
|
21
|
+
COPY lib/net/ssh/version.rb $INSTALL_PATH/lib/net/ssh/version.rb
|
|
22
|
+
|
|
23
|
+
RUN ls -l && gem install bundler && bundle install
|
|
24
|
+
|
|
25
|
+
COPY . $INSTALL_PATH/
|
|
26
|
+
|
|
27
|
+
CMD openssl version && ruby -ropenssl -e 'puts OpenSSL::OPENSSL_VERSION' && \
|
|
28
|
+
rpm -q zlib-ng && ruby -rzlib -e 'puts Zlib.zlib_version' && \
|
|
29
|
+
rake test
|