[INFO] fetching crate nexo-retailer-protocol 0.1.0...
[INFO] checking nexo-retailer-protocol-0.1.0 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate nexo-retailer-protocol 0.1.0 into /workspace/builds/worker-5-tc2/source
[INFO] removed /workspace/builds/worker-5-tc2/source/.cargo/config.toml
[INFO] started tweaking crates.io crate nexo-retailer-protocol 0.1.0
[INFO] removed 0 missing examples
[INFO] removed 0 missing tests
[INFO] finished tweaking crates.io crate nexo-retailer-protocol 0.1.0
[INFO] tweaked toml for crates.io crate nexo-retailer-protocol 0.1.0 written to /workspace/builds/worker-5-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate nexo-retailer-protocol 0.1.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 nexo-retailer-protocol 0.1.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] 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] 46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934
[INFO] running `Command { std: "docker" "start" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934", 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" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934", 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" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling getrandom v0.4.1
[INFO] [stderr]    Compiling libc v0.2.182
[INFO] [stderr]    Compiling anyhow v1.0.102
[INFO] [stderr]    Compiling itertools v0.14.0
[INFO] [stderr]    Compiling zerocopy v0.8.33
[INFO] [stderr]    Compiling syn v2.0.117
[INFO] [stderr]     Checking regex-automata v0.4.14
[INFO] [stderr]    Compiling bitflags v2.11.0
[INFO] [stderr]    Compiling regex-syntax v0.8.10
[INFO] [stderr]    Compiling bytes v1.11.1
[INFO] [stderr]    Compiling prettyplease v0.2.37
[INFO] [stderr]    Compiling linux-raw-sys v0.12.1
[INFO] [stderr]    Compiling indexmap v2.13.0
[INFO] [stderr]    Compiling hashbrown v0.15.5
[INFO] [stderr]    Compiling rustix v1.1.4
[INFO] [stderr]     Checking getrandom v0.3.4
[INFO] [stderr]     Checking ppv-lite86 v0.2.21
[INFO] [stderr]    Compiling fastrand v2.3.0
[INFO] [stderr]    Compiling fixedbitset v0.5.7
[INFO] [stderr]    Compiling once_cell v1.21.3
[INFO] [stderr]     Checking rand_core v0.9.5
[INFO] [stderr]     Checking getrandom v0.2.17
[INFO] [stderr]    Compiling multimap v0.10.1
[INFO] [stderr]    Compiling petgraph v0.8.3
[INFO] [stderr]     Checking rand_core v0.6.4
[INFO] [stderr]     Checking socket2 v0.6.2
[INFO] [stderr]     Checking mio v1.1.1
[INFO] [stderr]     Checking futures-channel v0.3.32
[INFO] [stderr]     Checking futures-io v0.3.32
[INFO] [stderr]    Compiling syn v1.0.109
[INFO] [stderr]     Checking tracing v0.1.44
[INFO] [stderr]     Checking matchers v0.2.0
[INFO] [stderr]     Checking regex v1.12.3
[INFO] [stderr]     Checking rand_chacha v0.3.1
[INFO] [stderr]     Checking wait-timeout v0.2.1
[INFO] [stderr]     Checking quick-error v1.2.3
[INFO] [stderr]     Checking bit-vec v0.8.0
[INFO] [stderr]     Checking rand v0.8.5
[INFO] [stderr]     Checking env_filter v1.0.0
[INFO] [stderr]    Compiling tempfile v3.26.0
[INFO] [stderr]     Checking bit-set v0.8.0
[INFO] [stderr]     Checking tracing-subscriber v0.3.22
[INFO] [stderr]     Checking rand v0.9.2
[INFO] [stderr]     Checking rand_xorshift v0.4.0
[INFO] [stderr]     Checking rusty-fork v0.3.1
[INFO] [stderr]     Checking rand_chacha v0.9.0
[INFO] [stderr]     Checking uuid v1.21.0
[INFO] [stderr]     Checking unarray v0.1.4
[INFO] [stderr]     Checking jiff v0.2.22
[INFO] [stderr]     Checking proptest v1.10.0
[INFO] [stderr]    Compiling prost-derive v0.14.3
[INFO] [stderr]    Compiling tokio-macros v2.6.0
[INFO] [stderr]    Compiling futures-macro v0.3.32
[INFO] [stderr]    Compiling async-trait v0.1.89
[INFO] [stderr]     Checking tokio v1.49.0
[INFO] [stderr]     Checking futures-util v0.3.32
[INFO] [stderr]    Compiling prost v0.14.3
[INFO] [stderr]     Checking prost-types v0.14.3
[INFO] [stderr]    Compiling prost-build v0.14.3
[INFO] [stderr]    Compiling proptest-derive v0.4.0
[INFO] [stderr]     Checking env_logger v0.11.9
[INFO] [stderr]     Checking futures-executor v0.3.32
[INFO] [stderr]     Checking futures v0.3.32
[INFO] [stderr]    Compiling nexo-retailer-protocol v0.1.0 (/opt/rustwide/workdir)
[INFO] [stderr]     Checking tokio-util v0.7.18
[INFO] [stdout] warning: unused import: `Ordering`
[INFO] [stdout]   --> src/client/timeout.rs:23:37
[INFO] [stdout]    |
[INFO] [stdout] 23 | use core::sync::atomic::{AtomicU64, Ordering};
[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: `alloc::string::String`
[INFO] [stdout]   --> src/error/mod.rs:32:5
[INFO] [stdout]    |
[INFO] [stdout] 32 | use alloc::string::String;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `core::time::Duration`
[INFO] [stdout]   --> src/transport/mod.rs:66:5
[INFO] [stdout]    |
[INFO] [stdout] 66 | use core::time::Duration;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `prost::alloc::string::ToString`
[INFO] [stdout]    --> src/client/builder.rs:706:13
[INFO] [stdout]     |
[INFO] [stdout] 706 |         use prost::alloc::string::ToString;
[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: `Ordering`
[INFO] [stdout]   --> src/client/timeout.rs:23:37
[INFO] [stdout]    |
[INFO] [stdout] 23 | use core::sync::atomic::{AtomicU64, Ordering};
[INFO] [stdout]    |                                     ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `alloc::string::String`
[INFO] [stdout]   --> src/error/mod.rs:32:5
[INFO] [stdout]    |
[INFO] [stdout] 32 | use alloc::string::String;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `core::time::Duration`
[INFO] [stdout]   --> src/transport/mod.rs:66:5
[INFO] [stdout]    |
[INFO] [stdout] 66 | use core::time::Duration;
[INFO] [stdout]    |     ^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: static `MESSAGE_ID_COUNTER` is never used
[INFO] [stdout]   --> src/client/timeout.rs:34:8
[INFO] [stdout]    |
[INFO] [stdout] 34 | static MESSAGE_ID_COUNTER: AtomicU64 = AtomicU64::new(0);
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `complete` is never used
[INFO] [stdout]   --> src/client/std.rs:69:8
[INFO] [stdout]    |
[INFO] [stdout] 36 | impl PendingRequests {
[INFO] [stdout]    | -------------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 69 |     fn complete(&mut self, id: String, response: Vec<u8>) -> Result<(), Vec<u8>> {
[INFO] [stdout]    |        ^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `DEFAULT_DEDUP_TTL` is never used
[INFO] [stdout]   --> src/server/connection.rs:14:7
[INFO] [stdout]    |
[INFO] [stdout] 14 | const DEFAULT_DEDUP_TTL: Duration = Duration::from_secs(5 * 60);
[INFO] [stdout]    |       ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `shutdown_flag` is never read
[INFO] [stdout]   --> src/server/std_impl.rs:59:5
[INFO] [stdout]    |
[INFO] [stdout] 53 | pub struct NexoServer {
[INFO] [stdout]    |            ---------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 59 |     shutdown_flag: Arc<tokio::sync::Notify>,
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `NexoError`
[INFO] [stdout]    --> src/transport/tokio.rs:466:17
[INFO] [stdout]     |
[INFO] [stdout] 466 | /                 NexoError::Connection {
[INFO] [stdout] 467 | |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 468 | |                 }
[INFO] [stdout] 469 | |                 .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] 468 -                 }
[INFO] [stdout] 469 -                 .into()
[INFO] [stdout] 468 +                 }
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 466 ~                 <NexoError as Into<_>>::into(NexoError::Connection {
[INFO] [stdout] 467 |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 468 ~                 })
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `NexoError`
[INFO] [stdout]    --> src/transport/tokio.rs:478:17
[INFO] [stdout]     |
[INFO] [stdout] 478 | /                 NexoError::Connection {
[INFO] [stdout] 479 | |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 480 | |                 }
[INFO] [stdout] 481 | |                 .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] 480 -                 }
[INFO] [stdout] 481 -                 .into()
[INFO] [stdout] 480 +                 }
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 478 ~                 <NexoError as Into<_>>::into(NexoError::Connection {
[INFO] [stdout] 479 |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 480 ~                 })
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:150:5
[INFO] [stdout]     |
[INFO] [stdout] 150 |     async fn read(&mut self, buf: &mut [u8]) -> Result<usize, Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]     = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 150 -     async fn read(&mut self, buf: &mut [u8]) -> Result<usize, Self::Error>;
[INFO] [stdout] 150 +     fn read(&mut self, buf: &mut [u8]) -> impl std::future::Future<Output = Result<usize, Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:168:5
[INFO] [stdout]     |
[INFO] [stdout] 168 |     async fn write(&mut self, buf: &[u8]) -> Result<usize, Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 168 -     async fn write(&mut self, buf: &[u8]) -> Result<usize, Self::Error>;
[INFO] [stdout] 168 +     fn write(&mut self, buf: &[u8]) -> impl std::future::Future<Output = Result<usize, Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:182:5
[INFO] [stdout]     |
[INFO] [stdout] 182 |     async fn connect(&mut self, addr: &str) -> Result<(), Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 182 -     async fn connect(&mut self, addr: &str) -> Result<(), Self::Error>;
[INFO] [stdout] 182 +     fn connect(&mut self, addr: &str) -> impl std::future::Future<Output = Result<(), Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `nexo-retailer-protocol` (lib) due to 2 previous errors; 10 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/client/std.rs:808:13
[INFO] [stdout]     |
[INFO] [stdout] 808 |         let mut client = NexoClient::with_transport(mock);
[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/heartbeat.rs:573:13
[INFO] [stdout]     |
[INFO] [stdout] 573 |         let mut monitor = HeartbeatMonitor::new(config);
[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/heartbeat.rs:652:13
[INFO] [stdout]     |
[INFO] [stdout] 652 |         let mut monitor = HeartbeatMonitor::new(config);
[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/transport/framing.rs:378:13
[INFO] [stdout]     |
[INFO] [stdout] 378 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:379:13
[INFO] [stdout]     |
[INFO] [stdout] 379 |         let mut framed = FramedTransport::new(transport);
[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/transport/framing.rs:381:13
[INFO] [stdout]     |
[INFO] [stdout] 381 |         let mut transport = MockTransport::new();
[INFO] [stdout]     |             ----^^^^^^^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `framed`
[INFO] [stdout]    --> src/transport/framing.rs:379:13
[INFO] [stdout]     |
[INFO] [stdout] 379 |         let mut framed = FramedTransport::new(transport);
[INFO] [stdout]     |             ^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_framed`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(unused_variables)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable does not need to be mutable
[INFO] [stdout]    --> src/transport/framing.rs:426:13
[INFO] [stdout]     |
[INFO] [stdout] 426 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:456:13
[INFO] [stdout]     |
[INFO] [stdout] 456 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:482:13
[INFO] [stdout]     |
[INFO] [stdout] 482 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:508:13
[INFO] [stdout]     |
[INFO] [stdout] 508 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:534:13
[INFO] [stdout]     |
[INFO] [stdout] 534 |         let mut transport = MockTransport::new().with_partial_reads(2);
[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/transport/framing.rs:560:13
[INFO] [stdout]     |
[INFO] [stdout] 560 |         let mut transport = MockTransport::new();
[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/transport/framing.rs:596:13
[INFO] [stdout]     |
[INFO] [stdout] 596 |         let mut transport = MockTransport::new();
[INFO] [stdout]     |             ----^^^^^^^^^
[INFO] [stdout]     |             |
[INFO] [stdout]     |             help: remove this `mut`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: static `MESSAGE_ID_COUNTER` is never used
[INFO] [stdout]   --> src/client/timeout.rs:34:8
[INFO] [stdout]    |
[INFO] [stdout] 34 | static MESSAGE_ID_COUNTER: AtomicU64 = AtomicU64::new(0);
[INFO] [stdout]    |        ^^^^^^^^^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `get_sent_data` is never used
[INFO] [stdout]    --> src/client/std.rs:759:12
[INFO] [stdout]     |
[INFO] [stdout] 746 |     impl MockFramedTransport {
[INFO] [stdout]     |     ------------------------ method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 759 |         fn get_sent_data(&self) -> &[u8] {
[INFO] [stdout]     |            ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: constant `DEFAULT_DEDUP_TTL` is never used
[INFO] [stdout]   --> src/server/connection.rs:14:7
[INFO] [stdout]    |
[INFO] [stdout] 14 | const DEFAULT_DEDUP_TTL: Duration = Duration::from_secs(5 * 60);
[INFO] [stdout]    |       ^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: method `reset` is never used
[INFO] [stdout]    --> src/server/handler.rs:321:16
[INFO] [stdout]     |
[INFO] [stdout] 309 |     impl MockRequestHandler {
[INFO] [stdout]     |     ----------------------- method in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 321 |         pub fn reset(&self) {
[INFO] [stdout]     |                ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `shutdown_flag` is never read
[INFO] [stdout]   --> src/server/std_impl.rs:59:5
[INFO] [stdout]    |
[INFO] [stdout] 53 | pub struct NexoServer {
[INFO] [stdout]    |            ---------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 59 |     shutdown_flag: Arc<tokio::sync::Notify>,
[INFO] [stdout]    |     ^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: function `get_available_port` is never used
[INFO] [stdout]    --> src/transport/tokio.rs:515:8
[INFO] [stdout]     |
[INFO] [stdout] 515 |     fn get_available_port() -> u16 {
[INFO] [stdout]     |        ^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `error::NexoError`
[INFO] [stdout]    --> src/transport/tokio.rs:466:17
[INFO] [stdout]     |
[INFO] [stdout] 466 | /                 NexoError::Connection {
[INFO] [stdout] 467 | |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 468 | |                 }
[INFO] [stdout] 469 | |                 .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] 468 -                 }
[INFO] [stdout] 469 -                 .into()
[INFO] [stdout] 468 +                 }
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 466 ~                 <error::NexoError as Into<_>>::into(NexoError::Connection {
[INFO] [stdout] 467 |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 468 ~                 })
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `error::NexoError`
[INFO] [stdout]    --> src/transport/tokio.rs:478:17
[INFO] [stdout]     |
[INFO] [stdout] 478 | /                 NexoError::Connection {
[INFO] [stdout] 479 | |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 480 | |                 }
[INFO] [stdout] 481 | |                 .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] 480 -                 }
[INFO] [stdout] 481 -                 .into()
[INFO] [stdout] 480 +                 }
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 478 ~                 <error::NexoError as Into<_>>::into(NexoError::Connection {
[INFO] [stdout] 479 |                     details: Box::leak(e.to_string().into_boxed_str()),
[INFO] [stdout] 480 ~                 })
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:150:5
[INFO] [stdout]     |
[INFO] [stdout] 150 |     async fn read(&mut self, buf: &mut [u8]) -> Result<usize, Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout]     = note: `#[warn(async_fn_in_trait)]` on by default
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 150 -     async fn read(&mut self, buf: &mut [u8]) -> Result<usize, Self::Error>;
[INFO] [stdout] 150 +     fn read(&mut self, buf: &mut [u8]) -> impl std::future::Future<Output = Result<usize, Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:168:5
[INFO] [stdout]     |
[INFO] [stdout] 168 |     async fn write(&mut self, buf: &[u8]) -> Result<usize, Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 168 -     async fn write(&mut self, buf: &[u8]) -> Result<usize, Self::Error>;
[INFO] [stdout] 168 +     fn write(&mut self, buf: &[u8]) -> impl std::future::Future<Output = Result<usize, Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: use of `async fn` in public traits is discouraged as auto trait bounds cannot be specified
[INFO] [stdout]    --> src/transport/mod.rs:182:5
[INFO] [stdout]     |
[INFO] [stdout] 182 |     async fn connect(&mut self, addr: &str) -> Result<(), Self::Error>;
[INFO] [stdout]     |     ^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: you can suppress this lint if you plan to use the trait only in your own code, or do not care about auto traits like `Send` on the `Future`
[INFO] [stdout] help: you can alternatively desugar to a normal `fn` that returns `impl Future` and add any desired bounds such as `Send`, but these cannot be relaxed without a breaking API change
[INFO] [stdout]     |
[INFO] [stdout] 182 -     async fn connect(&mut self, addr: &str) -> Result<(), Self::Error>;
[INFO] [stdout] 182 +     fn connect(&mut self, addr: &str) -> impl std::future::Future<Output = Result<(), Self::Error>> + Send;
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `nexo-retailer-protocol` (lib test) due to 2 previous errors; 27 warnings emitted
[INFO] [stderr] warning: unused dependency `tokio-util`
[INFO] [stderr]   --> Cargo.toml:54:15
[INFO] [stderr]    |
[INFO] [stderr] 54 | [dependencies.tokio-util]
[INFO] [stderr]    |               ^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `tokio-util`
[INFO] [stderr] warning: unused dependency `tracing-subscriber`
[INFO] [stderr]   --> Cargo.toml:65:15
[INFO] [stderr]    |
[INFO] [stderr] 65 | [dependencies.tracing-subscriber]
[INFO] [stderr]    |               ^^^^^^^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr] help: consider removing the dependency on `tracing-subscriber`
[INFO] [stderr] warning: `nexo-retailer-protocol` (manifest) generated 2 warnings
[INFO] running `Command { std: "docker" "inspect" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934", kill_on_drop: false }`
[INFO] [stdout] 46f604e21727ffc281f114eadee88289cf245eecc98159b7e605f9c3d64ea934
