[INFO] fetching crate lnk-thrussh 0.33.5...
[INFO] checking lnk-thrussh-0.33.5 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate lnk-thrussh 0.33.5 into /workspace/builds/worker-5-tc2/source
[INFO] started tweaking crates.io crate lnk-thrussh 0.33.5
[INFO] finished tweaking crates.io crate lnk-thrussh 0.33.5
[INFO] tweaked toml for crates.io crate lnk-thrussh 0.33.5 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate lnk-thrussh 0.33.5 on toolchain a5689b034ad287112f709693384a37f1377eb9cc
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate crates.io crate lnk-thrussh 0.33.5 already has a lockfile, it will not be regenerated
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-5-tc2/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/cargo-home:/opt/rustwide/cargo-home:ro,Z" "-v" "/var/lib/crater-agent-workspace/rustup-home:/opt/rustwide/rustup-home:ro,Z" "-m" "1610612736" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:77db811e55d90add9212f6832c23229e3de5f1c1f9905e8cb07319ea15df73ac" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b
[INFO] running `Command { std: "docker" "start" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-w" "/opt/rustwide/workdir" "--user" "0:0" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "exec" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_INCREMENTAL=0" "-e" "RUST_BACKTRACE=full" "-e" "RUSTFLAGS=--cap-lints=forbid" "-e" "RUSTDOCFLAGS=--cap-lints=forbid" "-w" "/opt/rustwide/workdir" "--user" "0:0" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling libc v0.2.119
[INFO] [stderr]    Compiling proc-macro2 v1.0.36
[INFO] [stderr]     Checking cfg-if v1.0.0
[INFO] [stderr]    Compiling unicode-xid v0.2.2
[INFO] [stderr]    Compiling typenum v1.15.0
[INFO] [stderr]    Compiling syn v1.0.86
[INFO] [stderr]    Compiling version_check v0.9.4
[INFO] [stderr]    Compiling autocfg v1.1.0
[INFO] [stderr]    Compiling memchr v2.4.1
[INFO] [stderr]    Compiling futures-core v0.3.21
[INFO] [stderr]    Compiling log v0.4.14
[INFO] [stderr]    Compiling futures-channel v0.3.21
[INFO] [stderr]     Checking pin-project-lite v0.2.8
[INFO] [stderr]    Compiling futures-task v0.3.21
[INFO] [stderr]    Compiling futures-util v0.3.21
[INFO] [stderr]     Checking subtle v2.4.1
[INFO] [stderr]     Checking futures-sink v0.3.21
[INFO] [stderr]    Compiling same-file v1.0.6
[INFO] [stderr]    Compiling winapi v0.3.9
[INFO] [stderr]     Checking opaque-debug v0.3.0
[INFO] [stderr]     Checking cpufeatures v0.2.1
[INFO] [stderr]    Compiling walkdir v2.3.2
[INFO] [stderr]     Checking byteorder v1.4.3
[INFO] [stderr]     Checking pin-utils v0.1.0
[INFO] [stderr]     Checking slab v0.4.5
[INFO] [stderr]    Compiling pkg-config v0.3.24
[INFO] [stderr]    Compiling cc v1.0.73
[INFO] [stderr]     Checking futures-io v0.3.21
[INFO] [stderr]    Compiling generic-array v0.14.5
[INFO] [stderr]     Checking once_cell v1.10.0
[INFO] [stderr]     Checking bytes v1.1.0
[INFO] [stderr]    Compiling async-trait v0.1.52
[INFO] [stderr]     Checking base64ct v1.3.3
[INFO] [stderr]    Compiling serde_derive v1.0.136
[INFO] [stderr]     Checking ppv-lite86 v0.2.16
[INFO] [stderr]    Compiling serde v1.0.136
[INFO] [stderr]    Compiling num-traits v0.2.14
[INFO] [stderr]    Compiling num-integer v0.1.44
[INFO] [stderr]    Compiling num-bigint v0.4.3
[INFO] [stderr]    Compiling miniz_oxide v0.4.4
[INFO] [stderr]    Compiling crc32fast v1.3.2
[INFO] [stderr]     Checking lazy_static v1.4.0
[INFO] [stderr]     Checking zeroize v1.3.0
[INFO] [stderr]     Checking block-padding v0.2.1
[INFO] [stderr]     Checking adler v1.0.2
[INFO] [stderr]     Checking data-encoding v2.3.2
[INFO] [stderr]     Checking bit-vec v0.6.3
[INFO] [stderr]    Compiling quote v1.0.15
[INFO] [stderr]    Compiling anyhow v1.0.55
[INFO] [stderr]     Checking regex-syntax v0.6.25
[INFO] [stderr]     Checking aho-corasick v0.7.18
[INFO] [stderr]     Checking md5 v0.7.0
[INFO] [stderr]     Checking quick-error v1.2.3
[INFO] [stderr]     Checking bitflags v1.3.2
[INFO] [stderr]     Checking humantime v1.3.0
[INFO] [stderr]     Checking termcolor v1.1.3
[INFO] [stderr]     Checking getrandom v0.2.5
[INFO] [stderr]     Checking num_cpus v1.13.1
[INFO] [stderr]     Checking signal-hook-registry v1.4.0
[INFO] [stderr]     Checking mio v0.8.0
[INFO] [stderr]     Checking socket2 v0.4.4
[INFO] [stderr]     Checking lnk-cryptovec v0.6.1
[INFO] [stderr]     Checking dirs-sys v0.3.6
[INFO] [stderr]     Checking atty v0.2.14
[INFO] [stderr]     Checking rand_core v0.6.3
[INFO] [stderr]     Checking dirs v3.0.2
[INFO] [stderr]     Checking password-hash v0.2.3
[INFO] [stderr]     Checking rand_chacha v0.3.1
[INFO] [stderr]    Compiling libsodium-sys v0.2.7
[INFO] [stderr]     Checking flate2 v1.0.22
[INFO] [stderr]     Checking rand v0.8.5
[INFO] [stderr]     Checking cipher v0.3.0
[INFO] [stderr]     Checking digest v0.9.0
[INFO] [stderr]     Checking block-buffer v0.9.0
[INFO] [stderr]     Checking crypto-mac v0.11.1
[INFO] [stderr]     Checking hmac v0.11.0
[INFO] [stderr]     Checking sha2 v0.9.9
[INFO] [stderr]     Checking ctr v0.8.0
[INFO] [stderr]     Checking blowfish v0.8.0
[INFO] [stderr]     Checking block-modes v0.8.1
[INFO] [stderr]     Checking aes v0.7.5
[INFO] [stderr]     Checking regex v1.5.4
[INFO] [stderr]     Checking yasna v0.4.0
[INFO] [stderr]     Checking pbkdf2 v0.8.0
[INFO] [stderr]     Checking bcrypt-pbkdf v0.6.2
[INFO] [stderr]     Checking env_logger v0.7.1
[INFO] [stderr]    Compiling futures-macro v0.3.21
[INFO] [stderr]    Compiling tokio-macros v1.7.0
[INFO] [stderr]    Compiling thiserror-impl v1.0.30
[INFO] [stderr]     Checking tokio v1.17.0
[INFO] [stderr]     Checking thiserror v1.0.30
[INFO] [stderr]     Checking lnk-thrussh-encoding v0.1.0
[INFO] [stderr]     Checking futures-executor v0.3.21
[INFO] [stderr]     Checking tokio-stream v0.1.8
[INFO] [stderr]     Checking futures v0.3.21
[INFO] [stderr]     Checking lnk-thrussh-agent v0.1.0
[INFO] [stderr]     Checking lnk-thrussh-libsodium v0.2.1
[INFO] [stderr]     Checking lnk-thrussh-keys v0.21.0
[INFO] [stderr]     Checking lnk-thrussh v0.33.5 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `self::kex::*`
[INFO] [stdout]   --> src/server/mod.rs:33:9
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub use self::kex::*;
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:170:44
[INFO] [stdout]     |
[INFO] [stdout] 170 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:173:44
[INFO] [stdout]     |
[INFO] [stdout] 173 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `self::kex::*`
[INFO] [stdout]   --> src/server/mod.rs:33:9
[INFO] [stdout]    |
[INFO] [stdout] 33 | pub use self::kex::*;
[INFO] [stdout]    |         ^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `super::super::*`
[INFO] [stdout]   --> src/server/encrypted.rs:15:5
[INFO] [stdout]    |
[INFO] [stdout] 15 | use super::super::*;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u32::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:170:44
[INFO] [stdout]     |
[INFO] [stdout] 170 |         assert!(packet_length <= std::u32::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(deprecated)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of deprecated constant `std::u8::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/cipher/mod.rs:173:44
[INFO] [stdout]     |
[INFO] [stdout] 173 |         assert!(padding_length <= std::u8::MAX as usize);
[INFO] [stdout]     |                                            ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/server/encrypted.rs:41:13
[INFO] [stdout]    |
[INFO] [stdout] 41 |         let mut enc = self.common.encrypted.as_mut().unwrap();
[INFO] [stdout]    |             ----^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/server/encrypted.rs:122:13
[INFO] [stdout]     |
[INFO] [stdout] 122 |         let mut enc = self.common.encrypted.as_mut().unwrap();
[INFO] [stdout]     |             ----^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]   --> src/server/encrypted.rs:41:13
[INFO] [stdout]    |
[INFO] [stdout] 41 |         let mut enc = self.common.encrypted.as_mut().unwrap();
[INFO] [stdout]    |             ----^^^
[INFO] [stdout]    |             |
[INFO] [stdout]    |             help: remove this `mut`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_mut)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/server/encrypted.rs:122:13
[INFO] [stdout]     |
[INFO] [stdout] 122 |         let mut enc = self.common.encrypted.as_mut().unwrap();
[INFO] [stdout]     |             ----^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: fields `key` and `algo` are never read
[INFO] [stdout]    --> src/auth.rs:143:9
[INFO] [stdout]     |
[INFO] [stdout] 142 |     PublicKey {
[INFO] [stdout]     |     --------- fields in this variant
[INFO] [stdout] 143 |         key: CryptoVec,
[INFO] [stdout]     |         ^^^
[INFO] [stdout] 144 |         algo: CryptoVec,
[INFO] [stdout]     |         ^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `CurrentRequest` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `name` is never read
[INFO] [stdout]   --> src/cipher/mod.rs:24:9
[INFO] [stdout]    |
[INFO] [stdout] 23 | pub struct Cipher {
[INFO] [stdout]    |            ------ field in this struct
[INFO] [stdout] 24 |     pub name: Name,
[INFO] [stdout]    |         ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `wants_reply` is never read
[INFO] [stdout]   --> src/session.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 27 | pub(crate) struct Encrypted {
[INFO] [stdout]    |                   --------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 39 |     pub wants_reply: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Encrypted` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `received` is never read
[INFO] [stdout]    --> src/session.rs:508:9
[INFO] [stdout]     |
[INFO] [stdout] 501 | pub struct NewKeys {
[INFO] [stdout]     |            ------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 508 |     pub received: bool,
[INFO] [stdout]     |         ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `NewKeys` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:136:24
[INFO] [stdout]     |
[INFO] [stdout] 136 |             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 136 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 136 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:148:24
[INFO] [stdout]     |
[INFO] [stdout] 148 |             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 148 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 148 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:159:24
[INFO] [stdout]     |
[INFO] [stdout] 159 |             return Err(Error::NoCommonCipher.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 159 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 159 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 159 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 159 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:173:28
[INFO] [stdout]     |
[INFO] [stdout] 173 |                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 173 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 173 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 173 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 173 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:181:28
[INFO] [stdout]     |
[INFO] [stdout] 181 |                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 181 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 181 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 181 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 181 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:201:22
[INFO] [stdout]     |
[INFO] [stdout] 201 |             _ => Err(Error::KexInit.into()),
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 201 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 201 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 201 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 201 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/server/kex.rs:42:28
[INFO] [stdout]    |
[INFO] [stdout] 42 |                 return Err(Error::UnknownKey.into());
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 42 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 42 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 42 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 42 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]    |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]    |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error`
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout]     |                                                       ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `lnk-thrussh` (lib) due to 16 previous errors; 10 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: fields `key` and `algo` are never read
[INFO] [stdout]    --> src/auth.rs:143:9
[INFO] [stdout]     |
[INFO] [stdout] 142 |     PublicKey {
[INFO] [stdout]     |     --------- fields in this variant
[INFO] [stdout] 143 |         key: CryptoVec,
[INFO] [stdout]     |         ^^^
[INFO] [stdout] 144 |         algo: CryptoVec,
[INFO] [stdout]     |         ^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `CurrentRequest` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]     = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `name` is never read
[INFO] [stdout]   --> src/cipher/mod.rs:24:9
[INFO] [stdout]    |
[INFO] [stdout] 23 | pub struct Cipher {
[INFO] [stdout]    |            ------ field in this struct
[INFO] [stdout] 24 |     pub name: Name,
[INFO] [stdout]    |         ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `wants_reply` is never read
[INFO] [stdout]   --> src/session.rs:39:9
[INFO] [stdout]    |
[INFO] [stdout] 27 | pub(crate) struct Encrypted {
[INFO] [stdout]    |                   --------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 39 |     pub wants_reply: bool,
[INFO] [stdout]    |         ^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `Encrypted` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `received` is never read
[INFO] [stdout]    --> src/session.rs:508:9
[INFO] [stdout]     |
[INFO] [stdout] 501 | pub struct NewKeys {
[INFO] [stdout]     |            ------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 508 |     pub received: bool,
[INFO] [stdout]     |         ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `NewKeys` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `client_pubkey` is never read
[INFO] [stdout]    --> src/lib.rs:718:9
[INFO] [stdout]     |
[INFO] [stdout] 717 |     struct Server {
[INFO] [stdout]     |            ------ field in this struct
[INFO] [stdout] 718 |         client_pubkey: Arc<lnk_thrussh_keys::key::PublicKey>,
[INFO] [stdout]     |         ^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `Server` has a derived impl for the trait `Clone`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:136:24
[INFO] [stdout]     |
[INFO] [stdout] 136 |             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout]     = note: `#[deny(self_type_conversion)]` on by default
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 136 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 136 +             return Err(Error::NoCommonKexAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 136 -             return Err(Error::NoCommonKexAlgo.into());
[INFO] [stdout] 136 +             return Err(<Error as Into<_>>::into(Error::NoCommonKexAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:148:24
[INFO] [stdout]     |
[INFO] [stdout] 148 |             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 148 +             return Err(Error::NoCommonKeyAlgo);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 148 -             return Err(Error::NoCommonKeyAlgo.into());
[INFO] [stdout] 148 +             return Err(<Error as Into<_>>::into(Error::NoCommonKeyAlgo));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:159:24
[INFO] [stdout]     |
[INFO] [stdout] 159 |             return Err(Error::NoCommonCipher.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 159 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 159 +             return Err(Error::NoCommonCipher);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 159 -             return Err(Error::NoCommonCipher.into());
[INFO] [stdout] 159 +             return Err(<Error as Into<_>>::into(Error::NoCommonCipher));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:173:28
[INFO] [stdout]     |
[INFO] [stdout] 173 |                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 173 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 173 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 173 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 173 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:181:28
[INFO] [stdout]     |
[INFO] [stdout] 181 |                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 181 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 181 +                 return Err(Error::NoCommonCompression);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 181 -                 return Err(Error::NoCommonCompression.into());
[INFO] [stdout] 181 +                 return Err(<Error as Into<_>>::into(Error::NoCommonCompression));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/negotiation.rs:201:22
[INFO] [stdout]     |
[INFO] [stdout] 201 |             _ => Err(Error::KexInit.into()),
[INFO] [stdout]     |                      ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 201 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 201 +             _ => Err(Error::KexInit),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 201 -             _ => Err(Error::KexInit.into()),
[INFO] [stdout] 201 +             _ => Err(<Error as Into<_>>::into(Error::KexInit)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/ssh_read.rs:125:28
[INFO] [stdout]     |
[INFO] [stdout] 125 |                 return Err(Error::Disconnect.into());
[INFO] [stdout]     |                            ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 125 -                 return Err(Error::Disconnect.into());
[INFO] [stdout] 125 +                 return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/server/kex.rs:42:28
[INFO] [stdout]    |
[INFO] [stdout] 42 |                 return Err(Error::UnknownKey.into());
[INFO] [stdout]    |                            ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 42 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 42 +                 return Err(Error::UnknownKey);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 42 -                 return Err(Error::UnknownKey.into());
[INFO] [stdout] 42 +                 return Err(<Error as Into<_>>::into(Error::UnknownKey));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:29:33
[INFO] [stdout]    |
[INFO] [stdout] 29 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]    |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 29 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 29 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:32:24
[INFO] [stdout]    |
[INFO] [stdout] 32 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 32 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 32 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:69:33
[INFO] [stdout]    |
[INFO] [stdout] 69 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]    |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 69 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 69 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]   --> src/client/session.rs:72:24
[INFO] [stdout]    |
[INFO] [stdout] 72 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]    |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(Error::Inconsistent);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 72 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 72 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:113:33
[INFO] [stdout]     |
[INFO] [stdout] 113 |                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout]     |                                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(Error::NotAuthenticated),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 113 -                 _ => return Err(Error::NotAuthenticated.into()),
[INFO] [stdout] 113 +                 _ => return Err(<Error as Into<_>>::into(Error::NotAuthenticated)),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/session.rs:116:24
[INFO] [stdout]     |
[INFO] [stdout] 116 |             return Err(Error::Inconsistent.into());
[INFO] [stdout]     |                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(Error::Inconsistent);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 116 -             return Err(Error::Inconsistent.into());
[INFO] [stdout] 116 +             return Err(<Error as Into<_>>::into(Error::Inconsistent));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `<S as Signer>::Error`
[INFO] [stdout]    --> src/client/mod.rs:292:55
[INFO] [stdout]     |
[INFO] [stdout] 292 |                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout]     |                                                       ^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(e)),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 292 -                         Err(e) => return (future, Err(e.into())),
[INFO] [stdout] 292 +                         Err(e) => return (future, Err(<<S as Signer>::Error as Into<_>>::into(e))),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `Error`
[INFO] [stdout]    --> src/client/mod.rs:326:32
[INFO] [stdout]     |
[INFO] [stdout] 326 |                     return Err(Error::Disconnect.into());
[INFO] [stdout]     |                                ^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: this method call relies on the `impl<T> Into<T> for T` blanket implementation and type inference, which is a semver hazard as a new `impl Into<_>` that affects your type might be added in the future causing type inference errors
[INFO] [stdout] help: consider removing the conversion call
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(Error::Disconnect);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 326 -                     return Err(Error::Disconnect.into());
[INFO] [stdout] 326 +                     return Err(<Error as Into<_>>::into(Error::Disconnect));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `lnk-thrussh` (lib test) due to 16 previous errors; 11 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b", kill_on_drop: false }`
[INFO] [stdout] e9043df4139b67cf25295f6167057e4400a0250a02cd4a2b18ee61d40c5d378b
