[INFO] fetching crate chatd 0.2.0...
[INFO] checking chatd-0.2.0 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate chatd 0.2.0 into /workspace/builds/worker-3-tc2/source
[INFO] started tweaking crates.io crate chatd 0.2.0
[INFO] finished tweaking crates.io crate chatd 0.2.0
[INFO] tweaked toml for crates.io crate chatd 0.2.0 written to /workspace/builds/worker-3-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate chatd 0.2.0 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 chatd 0.2.0 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] warning: `package.homepage` is redundant with `package.repository`
[INFO] [stderr]    --> Cargo.toml:111:12
[INFO] [stderr]     |
[INFO] [stderr] 111 | homepage = "https://github.com/unrenamed/chatd"
[INFO] [stderr]     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 116 | repository = "https://github.com/unrenamed/chatd"
[INFO] [stderr]     |              ------------------------------------
[INFO] [stderr]     |
[INFO] [stderr]     = note: `cargo::redundant_homepage` is set to `warn` by default
[INFO] [stderr] help: consider removing `package.homepage`
[INFO] [stderr] warning: `chatd` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-3-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] 060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd
[INFO] running `Command { std: "docker" "start" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd", 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" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd", 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" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr] warning: `package.homepage` is redundant with `package.repository`
[INFO] [stderr]    --> Cargo.toml:111:12
[INFO] [stderr]     |
[INFO] [stderr] 111 | homepage = "https://github.com/unrenamed/chatd"
[INFO] [stderr]     |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 116 | repository = "https://github.com/unrenamed/chatd"
[INFO] [stderr]     |              ------------------------------------
[INFO] [stderr]     |
[INFO] [stderr]     = note: `cargo::redundant_homepage` is set to `warn` by default
[INFO] [stderr] help: consider removing `package.homepage`
[INFO] [stderr] warning: `chatd` (manifest) generated 1 warning
[INFO] [stderr]    Compiling libc v0.2.155
[INFO] [stderr]     Checking zeroize v1.8.1
[INFO] [stderr]    Compiling proc-macro2 v1.0.86
[INFO] [stderr]    Compiling unicode-ident v1.0.12
[INFO] [stderr]    Compiling typenum v1.17.0
[INFO] [stderr]     Checking subtle v2.6.1
[INFO] [stderr]     Checking const-oid v0.9.6
[INFO] [stderr]     Checking cpufeatures v0.2.12
[INFO] [stderr]    Compiling autocfg v1.3.0
[INFO] [stderr]     Checking base64ct v1.6.0
[INFO] [stderr]    Compiling serde v1.0.205
[INFO] [stderr]    Compiling libm v0.2.8
[INFO] [stderr]     Checking smallvec v1.13.2
[INFO] [stderr]    Compiling generic-array v0.14.7
[INFO] [stderr]    Compiling semver v1.0.23
[INFO] [stderr]     Checking base16ct v0.2.0
[INFO] [stderr]    Compiling crossbeam-utils v0.8.20
[INFO] [stderr]     Checking bitflags v2.6.0
[INFO] [stderr]     Checking pin-project-lite v0.2.14
[INFO] [stderr]     Checking futures-core v0.3.30
[INFO] [stderr]     Checking opaque-debug v0.3.1
[INFO] [stderr]     Checking futures-sink v0.3.30
[INFO] [stderr]     Checking once_cell v1.19.0
[INFO] [stderr]     Checking futures-channel v0.3.30
[INFO] [stderr]     Checking pem-rfc7468 v0.7.0
[INFO] [stderr]     Checking aho-corasick v1.1.3
[INFO] [stderr]     Checking futures-io v0.3.30
[INFO] [stderr]     Checking futures-task v0.3.30
[INFO] [stderr]     Checking anstyle v1.0.8
[INFO] [stderr]     Checking regex-syntax v0.8.4
[INFO] [stderr]     Checking spin v0.9.8
[INFO] [stderr]    Compiling num-bigint-dig v0.8.4
[INFO] [stderr]    Compiling rustix v0.38.34
[INFO] [stderr]     Checking lazy_static v1.5.0
[INFO] [stderr]    Compiling thiserror v1.0.63
[INFO] [stderr]     Checking der v0.7.9
[INFO] [stderr]     Checking hashbrown v0.14.5
[INFO] [stderr]    Compiling rustversion v1.0.17
[INFO] [stderr]     Checking option-ext v0.2.0
[INFO] [stderr]     Checking linux-raw-sys v0.4.14
[INFO] [stderr]    Compiling signal-hook v0.3.17
[INFO] [stderr]    Compiling num-traits v0.2.19
[INFO] [stderr]    Compiling slab v0.4.9
[INFO] [stderr]    Compiling lock_api v0.4.12
[INFO] [stderr]    Compiling rustc_version v0.4.0
[INFO] [stderr]     Checking bytes v1.7.1
[INFO] [stderr]     Checking anstyle-parse v0.2.5
[INFO] [stderr]     Checking bstr v1.10.0
[INFO] [stderr]     Checking crossbeam-epoch v0.9.18
[INFO] [stderr]    Compiling quote v1.0.36
[INFO] [stderr]    Compiling curve25519-dalek v4.1.3
[INFO] [stderr]     Checking equivalent v1.0.1
[INFO] [stderr]     Checking predicates-core v1.0.8
[INFO] [stderr]     Checking adler v1.0.2
[INFO] [stderr]     Checking ryu v1.0.18
[INFO] [stderr]    Compiling anyhow v1.0.86
[INFO] [stderr]     Checking anstyle-query v1.1.1
[INFO] [stderr]     Checking itoa v1.0.11
[INFO] [stderr]    Compiling serde_json v1.0.122
[INFO] [stderr]     Checking destructure_traitobject v0.2.0
[INFO] [stderr]    Compiling portable-atomic v1.7.0
[INFO] [stderr]    Compiling syn v2.0.72
[INFO] [stderr]     Checking colorchoice v1.0.2
[INFO] [stderr]     Checking indexmap v2.3.0
[INFO] [stderr]     Checking anstream v0.6.15
[INFO] [stderr]     Checking unsafe-any-ors v1.0.0
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]     Checking miniz_oxide v0.7.4
[INFO] [stderr]     Checking crossbeam-deque v0.8.5
[INFO] [stderr]     Checking raw-cpuid v11.1.0
[INFO] [stderr]     Checking getrandom v0.2.15
[INFO] [stderr]     Checking signal-hook-registry v1.4.2
[INFO] [stderr]     Checking parking_lot_core v0.9.10
[INFO] [stderr]     Checking rand_core v0.6.4
[INFO] [stderr]     Checking ff v0.13.0
[INFO] [stderr]     Checking socket2 v0.5.7
[INFO] [stderr]     Checking password-hash v0.4.2
[INFO] [stderr]     Checking dirs-sys v0.4.1
[INFO] [stderr]     Checking num-integer v0.1.46
[INFO] [stderr]     Checking parking_lot v0.12.3
[INFO] [stderr]     Checking dirs v5.0.1
[INFO] [stderr]     Checking group v0.13.0
[INFO] [stderr]     Checking crypto-common v0.1.6
[INFO] [stderr]     Checking block-buffer v0.10.4
[INFO] [stderr]     Checking block-padding v0.3.3
[INFO] [stderr]     Checking universal-hash v0.5.1
[INFO] [stderr]     Checking crypto-bigint v0.5.5
[INFO] [stderr]     Checking inout v0.1.3
[INFO] [stderr]     Checking num-iter v0.1.45
[INFO] [stderr]     Checking polyval v0.6.2
[INFO] [stderr]     Checking digest v0.10.7
[INFO] [stderr]     Checking aead v0.5.2
[INFO] [stderr]     Checking poly1305 v0.8.0
[INFO] [stderr]     Checking ordered-float v2.10.1
[INFO] [stderr]     Checking cipher v0.4.4
[INFO] [stderr]     Checking russh-cryptovec v0.7.3
[INFO] [stderr]     Checking crc32fast v1.4.2
[INFO] [stderr]     Checking clap_lex v0.7.2
[INFO] [stderr]     Checking ghash v0.5.1
[INFO] [stderr]     Checking data-encoding v2.6.0
[INFO] [stderr]    Compiling doc-comment v0.3.3
[INFO] [stderr]     Checking difflib v0.4.0
[INFO] [stderr]     Checking iana-time-zone v0.1.60
[INFO] [stderr]     Checking hmac v0.12.1
[INFO] [stderr]     Checking sha2 v0.10.8
[INFO] [stderr]     Checking signature v2.2.0
[INFO] [stderr]     Checking sha1 v0.10.6
[INFO] [stderr]     Checking termtree v0.4.1
[INFO] [stderr]     Checking pbkdf2 v0.12.2
[INFO] [stderr]     Checking hkdf v0.12.4
[INFO] [stderr]     Checking spki v0.7.3
[INFO] [stderr]     Checking rfc6979 v0.4.0
[INFO] [stderr]    Compiling mockall_derive v0.13.0
[INFO] [stderr]     Checking aes v0.8.4
[INFO] [stderr]     Checking salsa20 v0.10.2
[INFO] [stderr]     Checking cbc v0.1.2
[INFO] [stderr]     Checking ctr v0.9.2
[INFO] [stderr]     Checking blowfish v0.9.1
[INFO] [stderr]     Checking chacha20 v0.9.1
[INFO] [stderr]     Checking md5 v0.7.0
[INFO] [stderr]     Checking ssh-encoding v0.2.0
[INFO] [stderr]     Checking pbkdf2 v0.11.0
[INFO] [stderr]     Checking bcrypt-pbkdf v0.10.0
[INFO] [stderr]     Checking scrypt v0.11.0
[INFO] [stderr]     Checking clap_builder v4.5.13
[INFO] [stderr]     Checking predicates-tree v1.0.11
[INFO] [stderr]     Checking predicates v3.1.2
[INFO] [stderr]     Checking chrono v0.4.38
[INFO] [stderr]     Checking flate2 v1.0.31
[INFO] [stderr]     Checking dashmap v5.5.3
[INFO] [stderr]     Checking thread-id v4.2.2
[INFO] [stderr]     Checking typemap-ors v1.0.0
[INFO] [stderr]     Checking spinning_top v0.3.0
[INFO] [stderr]     Checking unic-common v0.9.0
[INFO] [stderr]     Checking log-mdc v0.1.0
[INFO] [stderr]     Checking futures-timer v3.0.3
[INFO] [stderr]     Checking humantime v2.1.0
[INFO] [stderr]     Checking unic-char-range v0.9.0
[INFO] [stderr]     Checking no-std-compat v0.4.1
[INFO] [stderr]     Checking arc-swap v1.7.1
[INFO] [stderr]     Checking nonzero_ext v0.3.0
[INFO] [stderr]     Checking hex-literal v0.4.1
[INFO] [stderr]     Checking fastrand v2.1.0
[INFO] [stderr]     Checking unic-char-property v0.9.0
[INFO] [stderr]     Checking unic-ucd-version v0.9.0
[INFO] [stderr]     Checking unicode-width v0.1.13
[INFO] [stderr]     Checking downcast v0.11.0
[INFO] [stderr]     Checking unicode-segmentation v1.11.0
[INFO] [stderr]     Checking circular-buffer v0.1.7
[INFO] [stderr]     Checking pkcs5 v0.7.1
[INFO] [stderr]     Checking regex-automata v0.4.7
[INFO] [stderr]     Checking aes-gcm v0.10.3
[INFO] [stderr]     Checking fragile v2.0.0
[INFO] [stderr]     Checking quanta v0.12.3
[INFO] [stderr]     Checking terminal-keycode v1.1.1
[INFO] [stderr]     Checking unic-emoji-char v0.9.0
[INFO] [stderr]     Checking ssh-cipher v0.2.0
[INFO] [stderr]     Checking tempfile v3.12.0
[INFO] [stderr]     Checking pkcs8 v0.10.2
[INFO] [stderr]     Checking sec1 v0.7.3
[INFO] [stderr]     Checking ed25519 v2.2.3
[INFO] [stderr]     Checking pkcs1 v0.7.5
[INFO] [stderr]     Checking elliptic-curve v0.13.8
[INFO] [stderr]     Checking primeorder v0.13.6
[INFO] [stderr]     Checking ecdsa v0.16.9
[INFO] [stderr]    Compiling derivative v2.2.0
[INFO] [stderr]     Checking p384 v0.13.0
[INFO] [stderr]     Checking p256 v0.13.2
[INFO] [stderr]     Checking p521 v0.13.3
[INFO] [stderr]     Checking regex v1.10.6
[INFO] [stderr]    Compiling serde_derive v1.0.205
[INFO] [stderr]    Compiling zerocopy-derive v0.7.35
[INFO] [stderr]    Compiling futures-macro v0.3.30
[INFO] [stderr]    Compiling curve25519-dalek-derive v0.1.1
[INFO] [stderr]    Compiling tokio-macros v2.4.0
[INFO] [stderr]    Compiling thiserror-impl v1.0.63
[INFO] [stderr]    Compiling async-trait v0.1.81
[INFO] [stderr]    Compiling strum_macros v0.26.4
[INFO] [stderr]    Compiling clap_derive v4.5.13
[INFO] [stderr]    Compiling enum_dispatch v0.3.13
[INFO] [stderr]     Checking zerocopy v0.7.35
[INFO] [stderr]     Checking futures-util v0.3.30
[INFO] [stderr]     Checking ppv-lite86 v0.2.20
[INFO] [stderr]     Checking clap v4.5.13
[INFO] [stderr]     Checking strum v0.26.3
[INFO] [stderr]     Checking rand_chacha v0.3.1
[INFO] [stderr]     Checking rand v0.8.5
[INFO] [stderr]     Checking ed25519-dalek v2.1.1
[INFO] [stderr]     Checking num-bigint v0.4.6
[INFO] [stderr]     Checking futures-executor v0.3.30
[INFO] [stderr]     Checking futures v0.3.30
[INFO] [stderr]     Checking mockall v0.13.0
[INFO] [stderr]     Checking governor v0.6.3
[INFO] [stderr]     Checking rsa v0.9.6
[INFO] [stderr]     Checking ssh-key v0.6.6
[INFO] [stderr]     Checking log v0.4.22
[INFO] [stderr]     Checking serde_yaml v0.9.34+deprecated
[INFO] [stderr]     Checking serde-value v0.7.0
[INFO] [stderr]     Checking mio v1.0.1
[INFO] [stderr]     Checking globset v0.4.14
[INFO] [stderr]     Checking ignore v0.4.22
[INFO] [stderr]     Checking tokio v1.39.2
[INFO] [stderr]     Checking signal-hook-mio v0.2.4
[INFO] [stderr]     Checking crossterm v0.28.1
[INFO] [stderr]     Checking globwalk v0.9.1
[INFO] [stderr]     Checking log4rs v1.3.0
[INFO] [stderr]     Checking assert_fs v1.1.2
[INFO] [stderr]     Checking tokio-stream v0.1.15
[INFO] [stderr]     Checking russh-keys v0.44.0
[INFO] [stderr]     Checking russh v0.44.0
[INFO] [stderr]     Checking chatd v0.2.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: use of deprecated module `std::usize`: all constants in this module replaced by associated constants on the type
[INFO] [stdout]    --> src/chat/room/room.rs:319:14
[INFO] [stdout]     |
[INFO] [stdout] 319 |     use std::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::usize::MAX`: replaced by the `MAX` associated constant on this type
[INFO] [stdout]    --> src/chat/room/room.rs:746:27
[INFO] [stdout]     |
[INFO] [stdout] 746 |             .focus(usize::MAX); // focus at least one user, any user, except the message author
[INFO] [stdout]     |                           ^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:71:34
[INFO] [stdout]    |
[INFO] [stdout] 71 |         self.trusted_keys.insert(key.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] 71 -         self.trusted_keys.insert(key.into());
[INFO] [stdout] 71 +         self.trusted_keys.insert(key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 71 -         self.trusted_keys.insert(key.into());
[INFO] [stdout] 71 +         self.trusted_keys.insert(<PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:75:35
[INFO] [stdout]    |
[INFO] [stdout] 75 |         self.trusted_keys.remove(&key.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] 75 -         self.trusted_keys.remove(&key.into());
[INFO] [stdout] 75 +         self.trusted_keys.remove(&key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 75 -         self.trusted_keys.remove(&key.into());
[INFO] [stdout] 75 +         self.trusted_keys.remove(&<PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:87:31
[INFO] [stdout]    |
[INFO] [stdout] 87 |         self.operators.insert(key.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] 87 -         self.operators.insert(key.into());
[INFO] [stdout] 87 +         self.operators.insert(key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 87 -         self.operators.insert(key.into());
[INFO] [stdout] 87 +         self.operators.insert(<PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:91:32
[INFO] [stdout]    |
[INFO] [stdout] 91 |         self.operators.remove(&key.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] 91 -         self.operators.remove(&key.into());
[INFO] [stdout] 91 +         self.operators.remove(&key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 91 -         self.operators.remove(&key.into());
[INFO] [stdout] 91 +         self.operators.remove(&<PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `russh_keys::key::PublicKey`
[INFO] [stdout]   --> src/pubkey.rs:64:14
[INFO] [stdout]    |
[INFO] [stdout] 64 |         Self(key.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] 64 -         Self(key.into())
[INFO] [stdout] 64 +         Self(key)
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 64 -         Self(key.into())
[INFO] [stdout] 64 +         Self(<russh_keys::key::PublicKey as Into<_>>::into(key))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:287:32
[INFO] [stdout]     |
[INFO] [stdout] 287 |                 if auth.is_op(&user.public_key().clone().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] 287 -                 if auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 287 +                 if auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 287 -                 if auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 287 +                 if auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:521:33
[INFO] [stdout]     |
[INFO] [stdout] 521 |                 if !auth.is_op(&user.public_key().clone().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] 521 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 521 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 521 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 521 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:571:33
[INFO] [stdout]     |
[INFO] [stdout] 571 |                 if !auth.is_op(&user.public_key().clone().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] 571 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 571 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 571 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 571 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:593:33
[INFO] [stdout]     |
[INFO] [stdout] 593 |                 if !auth.is_op(&user.public_key().clone().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] 593 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 593 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 593 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 593 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:619:33
[INFO] [stdout]     |
[INFO] [stdout] 619 |                 if !auth.is_op(&user.public_key().clone().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] 619 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 619 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 619 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 619 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:713:33
[INFO] [stdout]     |
[INFO] [stdout] 713 |                 if !auth.is_op(&user.public_key().clone().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] 713 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 713 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 713 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 713 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:738:33
[INFO] [stdout]     |
[INFO] [stdout] 738 |                 if !auth.is_op(&user.public_key().clone().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] 738 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 738 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 738 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 738 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:748:33
[INFO] [stdout]     |
[INFO] [stdout] 748 |                 if !auth.is_op(&user.public_key().clone().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] 748 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 748 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 748 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 748 +                 if !auth.is_op(&<PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:812:60
[INFO] [stdout]     |
[INFO] [stdout] 812 |                         Some(user) => auth.add_trusted_key(user.public_key().clone().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] 812 -                         Some(user) => auth.add_trusted_key(user.public_key().clone().into()),
[INFO] [stdout] 812 +                         Some(user) => auth.add_trusted_key(user.public_key().clone()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 812 -                         Some(user) => auth.add_trusted_key(user.public_key().clone().into()),
[INFO] [stdout] 812 +                         Some(user) => auth.add_trusted_key(<PubKey as Into<_>>::into(user.public_key().clone())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/auth/set.rs:35:17
[INFO] [stdout]    |
[INFO] [stdout] 35 |     pub fn iter(&self) -> TimedHashSetIter<T> {
[INFO] [stdout]    |                 ^^^^^     ^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 35 |     pub fn iter(&self) -> TimedHashSetIter<'_, T> {
[INFO] [stdout]    |                                            +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/room/room.rs:74:29
[INFO] [stdout]    |
[INFO] [stdout] 74 |     pub fn members_iter_mut(&mut self) -> IterMut<UserName, RoomMember> {
[INFO] [stdout]    |                             ^^^^^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                             |
[INFO] [stdout]    |                             the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 74 |     pub fn members_iter_mut(&mut self) -> IterMut<'_, UserName, RoomMember> {
[INFO] [stdout]    |                                                   +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/room/room.rs:78:25
[INFO] [stdout]    |
[INFO] [stdout] 78 |     pub fn members_iter(&self) -> Iter<UserName, RoomMember> {
[INFO] [stdout]    |                         ^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                         |
[INFO] [stdout]    |                         the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 78 |     pub fn members_iter(&self) -> Iter<'_, UserName, RoomMember> {
[INFO] [stdout]    |                                        +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function pointer comparisons do not produce meaningful results since their addresses are not guaranteed to be unique
[INFO] [stdout]    --> src/chat/user/theme.rs:117:5
[INFO] [stdout]     |
[INFO] [stdout] 113 | #[derive(Clone, Debug, PartialEq)]
[INFO] [stdout]     |                        --------- in this derive macro expansion
[INFO] [stdout] ...
[INFO] [stdout] 117 |     username_fg: fn(String) -> Color,
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the address of the same function can vary between different codegen units
[INFO] [stdout]     = note: furthermore, different functions could have the same address after being merged together
[INFO] [stdout]     = note: for more information visit <https://doc.rust-lang.org/nightly/core/ptr/fn.fn_addr_eq.html>
[INFO] [stdout]     = note: `#[warn(unpredictable_function_pointer_comparisons)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/message/message_history.rs:21:17
[INFO] [stdout]    |
[INFO] [stdout] 21 |     pub fn iter(&self) -> circular_buffer::Iter<Message> {
[INFO] [stdout]    |                 ^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 21 |     pub fn iter(&self) -> circular_buffer::Iter<'_, Message> {
[INFO] [stdout]    |                                                 +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `chatd` (bin "chatd") due to 14 previous errors; 5 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:71:34
[INFO] [stdout]    |
[INFO] [stdout] 71 |         self.trusted_keys.insert(key.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] 71 -         self.trusted_keys.insert(key.into());
[INFO] [stdout] 71 +         self.trusted_keys.insert(key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 71 -         self.trusted_keys.insert(key.into());
[INFO] [stdout] 71 +         self.trusted_keys.insert(<pubkey::PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:75:35
[INFO] [stdout]    |
[INFO] [stdout] 75 |         self.trusted_keys.remove(&key.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] 75 -         self.trusted_keys.remove(&key.into());
[INFO] [stdout] 75 +         self.trusted_keys.remove(&key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 75 -         self.trusted_keys.remove(&key.into());
[INFO] [stdout] 75 +         self.trusted_keys.remove(&<pubkey::PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:87:31
[INFO] [stdout]    |
[INFO] [stdout] 87 |         self.operators.insert(key.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] 87 -         self.operators.insert(key.into());
[INFO] [stdout] 87 +         self.operators.insert(key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 87 -         self.operators.insert(key.into());
[INFO] [stdout] 87 +         self.operators.insert(<pubkey::PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]   --> src/auth/auth.rs:91:32
[INFO] [stdout]    |
[INFO] [stdout] 91 |         self.operators.remove(&key.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] 91 -         self.operators.remove(&key.into());
[INFO] [stdout] 91 +         self.operators.remove(&key);
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 91 -         self.operators.remove(&key.into());
[INFO] [stdout] 91 +         self.operators.remove(&<pubkey::PubKey as Into<_>>::into(key));
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:251:34
[INFO] [stdout]     |
[INFO] [stdout] 251 |         assert!(auth.is_trusted(&pubkey.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] 251 -         assert!(auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 251 +         assert!(auth.is_trusted(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 251 -         assert!(auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 251 +         assert!(auth.is_trusted(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:273:29
[INFO] [stdout]     |
[INFO] [stdout] 273 |         assert!(auth.is_op(&pubkey.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] 273 -         assert!(auth.is_op(&pubkey.into()));
[INFO] [stdout] 273 +         assert!(auth.is_op(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 273 -         assert!(auth.is_op(&pubkey.into()));
[INFO] [stdout] 273 +         assert!(auth.is_op(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:294:30
[INFO] [stdout]     |
[INFO] [stdout] 294 |         auth.add_trusted_key(pubkey.clone().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] 294 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 294 +         auth.add_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 294 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 294 +         auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:338:27
[INFO] [stdout]     |
[INFO] [stdout] 338 |         auth.add_operator(pubkey.clone().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] 338 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 338 +         auth.add_operator(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 338 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 338 +         auth.add_operator(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:340:29
[INFO] [stdout]     |
[INFO] [stdout] 340 |         assert!(auth.is_op(&pubkey.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] 340 -         assert!(auth.is_op(&pubkey.into()));
[INFO] [stdout] 340 +         assert!(auth.is_op(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 340 -         assert!(auth.is_op(&pubkey.into()));
[INFO] [stdout] 340 +         assert!(auth.is_op(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:347:30
[INFO] [stdout]     |
[INFO] [stdout] 347 |         auth.add_trusted_key(pubkey.clone().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] 347 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 347 +         auth.add_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 347 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 347 +         auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:349:34
[INFO] [stdout]     |
[INFO] [stdout] 349 |         assert!(auth.is_trusted(&pubkey.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] 349 -         assert!(auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 349 +         assert!(auth.is_trusted(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 349 -         assert!(auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 349 +         assert!(auth.is_trusted(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:362:44
[INFO] [stdout]     |
[INFO] [stdout] 362 |         assert!(auth.check_bans(username, &pubkey.clone().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] 362 -         assert!(auth.check_bans(username, &pubkey.clone().into()));
[INFO] [stdout] 362 +         assert!(auth.check_bans(username, &pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 362 -         assert!(auth.check_bans(username, &pubkey.clone().into()));
[INFO] [stdout] 362 +         assert!(auth.check_bans(username, &<pubkey::PubKey as Into<_>>::into(pubkey.clone())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:363:38
[INFO] [stdout]     |
[INFO] [stdout] 363 |         assert!(auth.check_bans("", &pubkey.into())); // Check if fingerprint banning works
[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] 363 -         assert!(auth.check_bans("", &pubkey.into())); // Check if fingerprint banning works
[INFO] [stdout] 363 +         assert!(auth.check_bans("", &pubkey)); // Check if fingerprint banning works
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 363 -         assert!(auth.check_bans("", &pubkey.into())); // Check if fingerprint banning works
[INFO] [stdout] 363 +         assert!(auth.check_bans("", &<pubkey::PubKey as Into<_>>::into(pubkey))); // Check if fingerprint banning works
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:387:27
[INFO] [stdout]     |
[INFO] [stdout] 387 |         auth.add_operator(pubkey.clone().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] 387 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 387 +         auth.add_operator(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 387 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 387 +         auth.add_operator(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:388:29
[INFO] [stdout]     |
[INFO] [stdout] 388 |         assert!(auth.is_op(&pubkey.clone().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] 388 -         assert!(auth.is_op(&pubkey.clone().into()));
[INFO] [stdout] 388 +         assert!(auth.is_op(&pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 388 -         assert!(auth.is_op(&pubkey.clone().into()));
[INFO] [stdout] 388 +         assert!(auth.is_op(&<pubkey::PubKey as Into<_>>::into(pubkey.clone())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:390:30
[INFO] [stdout]     |
[INFO] [stdout] 390 |         auth.remove_operator(pubkey.clone().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] 390 -         auth.remove_operator(pubkey.clone().into());
[INFO] [stdout] 390 +         auth.remove_operator(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 390 -         auth.remove_operator(pubkey.clone().into());
[INFO] [stdout] 390 +         auth.remove_operator(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:391:30
[INFO] [stdout]     |
[INFO] [stdout] 391 |         assert!(!auth.is_op(&pubkey.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] 391 -         assert!(!auth.is_op(&pubkey.into()));
[INFO] [stdout] 391 +         assert!(!auth.is_op(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 391 -         assert!(!auth.is_op(&pubkey.into()));
[INFO] [stdout] 391 +         assert!(!auth.is_op(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:399:30
[INFO] [stdout]     |
[INFO] [stdout] 399 |         auth.add_trusted_key(pubkey.clone().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] 399 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 399 +         auth.add_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 399 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 399 +         auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:400:34
[INFO] [stdout]     |
[INFO] [stdout] 400 |         assert!(auth.is_trusted(&pubkey.clone().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] 400 -         assert!(auth.is_trusted(&pubkey.clone().into()));
[INFO] [stdout] 400 +         assert!(auth.is_trusted(&pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 400 -         assert!(auth.is_trusted(&pubkey.clone().into()));
[INFO] [stdout] 400 +         assert!(auth.is_trusted(&<pubkey::PubKey as Into<_>>::into(pubkey.clone())));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:402:33
[INFO] [stdout]     |
[INFO] [stdout] 402 |         auth.remove_trusted_key(pubkey.clone().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] 402 -         auth.remove_trusted_key(pubkey.clone().into());
[INFO] [stdout] 402 +         auth.remove_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 402 -         auth.remove_trusted_key(pubkey.clone().into());
[INFO] [stdout] 402 +         auth.remove_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:403:35
[INFO] [stdout]     |
[INFO] [stdout] 403 |         assert!(!auth.is_trusted(&pubkey.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] 403 -         assert!(!auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 403 +         assert!(!auth.is_trusted(&pubkey));
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 403 -         assert!(!auth.is_trusted(&pubkey.into()));
[INFO] [stdout] 403 +         assert!(!auth.is_trusted(&<pubkey::PubKey as Into<_>>::into(pubkey)));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:410:27
[INFO] [stdout]     |
[INFO] [stdout] 410 |         auth.add_operator(pubkey.clone().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] 410 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 410 +         auth.add_operator(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 410 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 410 +         auth.add_operator(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:419:30
[INFO] [stdout]     |
[INFO] [stdout] 419 |         auth.add_trusted_key(pubkey.clone().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] 419 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 419 +         auth.add_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 419 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 419 +         auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:428:27
[INFO] [stdout]     |
[INFO] [stdout] 428 |         auth.add_operator(pubkey.clone().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] 428 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 428 +         auth.add_operator(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 428 -         auth.add_operator(pubkey.clone().into());
[INFO] [stdout] 428 +         auth.add_operator(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/auth/auth.rs:436:30
[INFO] [stdout]     |
[INFO] [stdout] 436 |         auth.add_trusted_key(pubkey.clone().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] 436 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 436 +         auth.add_trusted_key(pubkey.clone());
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 436 -         auth.add_trusted_key(pubkey.clone().into());
[INFO] [stdout] 436 +         auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(pubkey.clone()));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `russh_keys::key::PublicKey`
[INFO] [stdout]   --> src/pubkey.rs:64:14
[INFO] [stdout]    |
[INFO] [stdout] 64 |         Self(key.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] 64 -         Self(key.into())
[INFO] [stdout] 64 +         Self(key)
[INFO] [stdout]    |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]    |
[INFO] [stdout] 64 -         Self(key.into())
[INFO] [stdout] 64 +         Self(<russh_keys::key::PublicKey as Into<_>>::into(key))
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:287:32
[INFO] [stdout]     |
[INFO] [stdout] 287 |                 if auth.is_op(&user.public_key().clone().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] 287 -                 if auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 287 +                 if auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 287 -                 if auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 287 +                 if auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:521:33
[INFO] [stdout]     |
[INFO] [stdout] 521 |                 if !auth.is_op(&user.public_key().clone().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] 521 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 521 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 521 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 521 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:571:33
[INFO] [stdout]     |
[INFO] [stdout] 571 |                 if !auth.is_op(&user.public_key().clone().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] 571 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 571 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 571 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 571 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:593:33
[INFO] [stdout]     |
[INFO] [stdout] 593 |                 if !auth.is_op(&user.public_key().clone().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] 593 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 593 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 593 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 593 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:619:33
[INFO] [stdout]     |
[INFO] [stdout] 619 |                 if !auth.is_op(&user.public_key().clone().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] 619 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 619 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 619 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 619 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:713:33
[INFO] [stdout]     |
[INFO] [stdout] 713 |                 if !auth.is_op(&user.public_key().clone().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] 713 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 713 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 713 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 713 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:738:33
[INFO] [stdout]     |
[INFO] [stdout] 738 |                 if !auth.is_op(&user.public_key().clone().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] 738 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 738 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 738 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 738 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:748:33
[INFO] [stdout]     |
[INFO] [stdout] 748 |                 if !auth.is_op(&user.public_key().clone().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] 748 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 748 +                 if !auth.is_op(&user.public_key().clone()) {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 748 -                 if !auth.is_op(&user.public_key().clone().into()) {
[INFO] [stdout] 748 +                 if !auth.is_op(&<pubkey::PubKey as Into<_>>::into(user.public_key().clone())) {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `pubkey::PubKey`
[INFO] [stdout]    --> src/server/session_workflow/command_exec.rs:812:60
[INFO] [stdout]     |
[INFO] [stdout] 812 |                         Some(user) => auth.add_trusted_key(user.public_key().clone().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] 812 -                         Some(user) => auth.add_trusted_key(user.public_key().clone().into()),
[INFO] [stdout] 812 +                         Some(user) => auth.add_trusted_key(user.public_key().clone()),
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 812 -                         Some(user) => auth.add_trusted_key(user.public_key().clone().into()),
[INFO] [stdout] 812 +                         Some(user) => auth.add_trusted_key(<pubkey::PubKey as Into<_>>::into(user.public_key().clone())),
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/auth/set.rs:35:17
[INFO] [stdout]    |
[INFO] [stdout] 35 |     pub fn iter(&self) -> TimedHashSetIter<T> {
[INFO] [stdout]    |                 ^^^^^     ^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout]    = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 35 |     pub fn iter(&self) -> TimedHashSetIter<'_, T> {
[INFO] [stdout]    |                                            +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/room/room.rs:74:29
[INFO] [stdout]    |
[INFO] [stdout] 74 |     pub fn members_iter_mut(&mut self) -> IterMut<UserName, RoomMember> {
[INFO] [stdout]    |                             ^^^^^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                             |
[INFO] [stdout]    |                             the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 74 |     pub fn members_iter_mut(&mut self) -> IterMut<'_, UserName, RoomMember> {
[INFO] [stdout]    |                                                   +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/room/room.rs:78:25
[INFO] [stdout]    |
[INFO] [stdout] 78 |     pub fn members_iter(&self) -> Iter<UserName, RoomMember> {
[INFO] [stdout]    |                         ^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                         |
[INFO] [stdout]    |                         the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 78 |     pub fn members_iter(&self) -> Iter<'_, UserName, RoomMember> {
[INFO] [stdout]    |                                        +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function pointer comparisons do not produce meaningful results since their addresses are not guaranteed to be unique
[INFO] [stdout]    --> src/chat/user/theme.rs:117:5
[INFO] [stdout]     |
[INFO] [stdout] 113 | #[derive(Clone, Debug, PartialEq)]
[INFO] [stdout]     |                        --------- in this derive macro expansion
[INFO] [stdout] ...
[INFO] [stdout] 117 |     username_fg: fn(String) -> Color,
[INFO] [stdout]     |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: the address of the same function can vary between different codegen units
[INFO] [stdout]     = note: furthermore, different functions could have the same address after being merged together
[INFO] [stdout]     = note: for more information visit <https://doc.rust-lang.org/nightly/core/ptr/fn.fn_addr_eq.html>
[INFO] [stdout]     = note: `#[warn(unpredictable_function_pointer_comparisons)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: hiding a lifetime that's elided elsewhere is confusing
[INFO] [stdout]   --> src/chat/message/message_history.rs:21:17
[INFO] [stdout]    |
[INFO] [stdout] 21 |     pub fn iter(&self) -> circular_buffer::Iter<Message> {
[INFO] [stdout]    |                 ^^^^^     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the same lifetime is hidden here
[INFO] [stdout]    |                 |
[INFO] [stdout]    |                 the lifetime is elided here
[INFO] [stdout]    |
[INFO] [stdout]    = help: the same lifetime is referred to in inconsistent ways, making the signature confusing
[INFO] [stdout] help: use `'_` for type paths
[INFO] [stdout]    |
[INFO] [stdout] 21 |     pub fn iter(&self) -> circular_buffer::Iter<'_, Message> {
[INFO] [stdout]    |                                                 +++
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `chatd` (bin "chatd" test) due to 35 previous errors; 7 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd", kill_on_drop: false }`
[INFO] [stdout] 060cc83c14f528d1286185ad7dac04afbc0a20fe262645201ec078c894e30ffd
