[INFO] fetching crate jsonrpc-server-utils 18.0.0...
[INFO] checking jsonrpc-server-utils-18.0.0 against a5689b034ad287112f709693384a37f1377eb9cc for pr-129249-3
[INFO] extracting crate jsonrpc-server-utils 18.0.0 into /workspace/builds/worker-1-tc2/source
[INFO] started tweaking crates.io crate jsonrpc-server-utils 18.0.0
[INFO] finished tweaking crates.io crate jsonrpc-server-utils 18.0.0
[INFO] tweaked toml for crates.io crate jsonrpc-server-utils 18.0.0 written to /workspace/builds/worker-1-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate jsonrpc-server-utils 18.0.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] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "generate-lockfile" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Updating crates.io index
[INFO] [stderr]      Locking 41 packages to highest compatible versions
[INFO] [stderr]       Adding tokio-util v0.6.10 (available: v0.7.20)
[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:43:12
[INFO] [stderr]    |
[INFO] [stderr] 43 | homepage = "https://github.com/paritytech/jsonrpc"
[INFO] [stderr]    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 47 | repository = "https://github.com/paritytech/jsonrpc"
[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: `jsonrpc-server-utils` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-1-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] 9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb
[INFO] running `Command { std: "docker" "start" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb", 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" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb" "/opt/rustwide/cargo-home/bin/cargo" "+a5689b034ad287112f709693384a37f1377eb9cc" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb", 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" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb" "/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:43:12
[INFO] [stderr]    |
[INFO] [stderr] 43 | homepage = "https://github.com/paritytech/jsonrpc"
[INFO] [stderr]    |            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stderr] ...
[INFO] [stderr] 47 | repository = "https://github.com/paritytech/jsonrpc"
[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: `jsonrpc-server-utils` (manifest) generated 1 warning
[INFO] [stderr]    Compiling proc-macro2 v1.0.107
[INFO] [stderr]     Checking futures-channel v0.3.34
[INFO] [stderr]     Checking socket2 v0.6.5
[INFO] [stderr]     Checking mio v1.2.4
[INFO] [stderr]     Checking serde v1.0.229
[INFO] [stderr]     Checking regex-automata v0.4.18
[INFO] [stderr]     Checking unicase v2.10.0
[INFO] [stderr]     Checking tokio v1.53.3
[INFO] [stderr]    Compiling quote v1.0.47
[INFO] [stderr]    Compiling syn v3.0.9
[INFO] [stderr]     Checking globset v0.4.20
[INFO] [stderr]    Compiling futures-macro v0.3.34
[INFO] [stderr]    Compiling serde_derive v1.0.229
[INFO] [stderr]     Checking tokio-util v0.6.10
[INFO] [stderr]     Checking tokio-stream v0.1.19
[INFO] [stderr]     Checking futures-util v0.3.34
[INFO] [stderr]     Checking futures-executor v0.3.34
[INFO] [stderr]     Checking futures v0.3.34
[INFO] [stderr]     Checking jsonrpc-core v18.0.0
[INFO] [stderr]     Checking jsonrpc-server-utils v18.0.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: field `executor` is never read
[INFO] [stdout]   --> src/reactor.rs:78:2
[INFO] [stdout]    |
[INFO] [stdout] 77 | pub struct RpcEventLoop {
[INFO] [stdout]    |            ------------ field in this struct
[INFO] [stdout] 78 |     executor: TaskExecutor,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RpcEventLoop` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `executor` is never read
[INFO] [stdout]   --> src/reactor.rs:78:2
[INFO] [stdout]    |
[INFO] [stdout] 77 | pub struct RpcEventLoop {
[INFO] [stdout]    |            ------------ field in this struct
[INFO] [stdout] 78 |     executor: TaskExecutor,
[INFO] [stdout]    |     ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `RpcEventLoop` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:425:21
[INFO] [stdout]     |
[INFO] [stdout] 425 |         let origin = Some("http://parity.io".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] 425 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 425 +         let origin = Some("http://parity.io");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 425 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 425 +         let origin = Some(<&str as Into<_>>::into("http://parity.io"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:442:21
[INFO] [stdout]     |
[INFO] [stdout] 442 |         let origin = Some("http://parity.io".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] 442 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 442 +         let origin = Some("http://parity.io");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 442 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 442 +         let origin = Some(<&str as Into<_>>::into("http://parity.io"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:471:21
[INFO] [stdout]     |
[INFO] [stdout] 471 |         let origin = Some("null".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] 471 -         let origin = Some("null".into());
[INFO] [stdout] 471 +         let origin = Some("null");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 471 -         let origin = Some("null".into());
[INFO] [stdout] 471 +         let origin = Some(<&str as Into<_>>::into("null"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:484:21
[INFO] [stdout]     |
[INFO] [stdout] 484 |         let origin = Some("http://parity.io".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] 484 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 484 +         let origin = Some("http://parity.io");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 484 -         let origin = Some("http://parity.io".into());
[INFO] [stdout] 484 +         let origin = Some(<&str as Into<_>>::into("http://parity.io"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:507:22
[INFO] [stdout]     |
[INFO] [stdout] 507 |         let origin1 = Some("http://parity.io".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] 507 -         let origin1 = Some("http://parity.io".into());
[INFO] [stdout] 507 +         let origin1 = Some("http://parity.io");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 507 -         let origin1 = Some("http://parity.io".into());
[INFO] [stdout] 507 +         let origin1 = Some(<&str as Into<_>>::into("http://parity.io"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:508:22
[INFO] [stdout]     |
[INFO] [stdout] 508 |         let origin2 = Some("http://parity.iot".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] 508 -         let origin2 = Some("http://parity.iot".into());
[INFO] [stdout] 508 +         let origin2 = Some("http://parity.iot");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 508 -         let origin2 = Some("http://parity.iot".into());
[INFO] [stdout] 508 +         let origin2 = Some(<&str as Into<_>>::into("http://parity.iot"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `&str`
[INFO] [stdout]    --> src/cors.rs:509:22
[INFO] [stdout]     |
[INFO] [stdout] 509 |         let origin3 = Some("chrome-extension://test".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] 509 -         let origin3 = Some("chrome-extension://test".into());
[INFO] [stdout] 509 +         let origin3 = Some("chrome-extension://test");
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 509 -         let origin3 = Some("chrome-extension://test".into());
[INFO] [stdout] 509 +         let origin3 = Some(<&str as Into<_>>::into("chrome-extension://test"));
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cors::AccessControlAllowHeaders`
[INFO] [stdout]    --> src/cors.rs:541:71
[INFO] [stdout]     |
[INFO] [stdout] 541 |         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| x);
[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] 541 -         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| x);
[INFO] [stdout] 541 +         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers, |x| x);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 541 -         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| x);
[INFO] [stdout] 541 +         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &<cors::AccessControlAllowHeaders as Into<_>>::into(cors_allow_headers), |x| x);
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cors::AccessControlAllowHeaders`
[INFO] [stdout]    --> src/cors.rs:556:71
[INFO] [stdout]     |
[INFO] [stdout] 556 |         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| {
[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] 556 -         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| {
[INFO] [stdout] 556 +         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers, |x| {
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 556 -         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &cors_allow_headers.into(), |x| {
[INFO] [stdout] 556 +         let res = get_cors_allow_headers(headers.iter(), requested.iter(), &<cors::AccessControlAllowHeaders as Into<_>>::into(cors_allow_headers), |x| {
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: useless conversion to the same type: `cors::AccessControlAllowHeaders`
[INFO] [stdout]    --> src/cors.rs:586:68
[INFO] [stdout]     |
[INFO] [stdout] 586 |         let res = get_cors_allow_headers(headers.iter(), iter::empty(), &cors_allow_headers.into(), |x| x);
[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] 586 -         let res = get_cors_allow_headers(headers.iter(), iter::empty(), &cors_allow_headers.into(), |x| x);
[INFO] [stdout] 586 +         let res = get_cors_allow_headers(headers.iter(), iter::empty(), &cors_allow_headers, |x| x);
[INFO] [stdout]     |
[INFO] [stdout] help: you can use the fully-qualified path to avoid triggering this lint
[INFO] [stdout]     |
[INFO] [stdout] 586 -         let res = get_cors_allow_headers(headers.iter(), iter::empty(), &cors_allow_headers.into(), |x| x);
[INFO] [stdout] 586 +         let res = get_cors_allow_headers(headers.iter(), iter::empty(), &<cors::AccessControlAllowHeaders as Into<_>>::into(cors_allow_headers), |x| x);
[INFO] [stdout]     |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `jsonrpc-server-utils` (lib test) due to 10 previous errors; 1 warning emitted
[INFO] [stderr] warning: unused dependency `jsonrpc-core`
[INFO] [stderr]   --> Cargo.toml:14:15
[INFO] [stderr]    |
[INFO] [stderr] 14 | [dependencies.jsonrpc-core]
[INFO] [stderr]    |               ^^^^^^^^^^^^^
[INFO] [stderr]    |
[INFO] [stderr]    = note: `cargo::unused_dependencies` is set to `warn` by default
[INFO] [stderr] help: consider removing the dependency on `jsonrpc-core`
[INFO] [stderr] warning: `jsonrpc-server-utils` (manifest) generated 1 warning
[INFO] running `Command { std: "docker" "inspect" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb", kill_on_drop: false }`
[INFO] [stdout] 9ae3869565faf2d2de0ff497d26d3e53257be839716160f8ec9c371e3217eacb
