[INFO] fetching crate jsonrpc-server-utils 18.0.0...
[INFO] checking jsonrpc-server-utils-18.0.0 against c60b92c03ca8a3b101eb9d57f51e2232690638c7 for pr-164148-crater-rollup
[INFO] extracting crate jsonrpc-server-utils 18.0.0 into /workspace/builds/worker-4-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-4-tc2/source/Cargo.toml
[INFO] validating manifest of crates.io crate jsonrpc-server-utils 18.0.0 on toolchain c60b92c03ca8a3b101eb9d57f51e2232690638c7
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "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" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "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" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "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-4-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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] d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c
[INFO] running `Command { std: "docker" "start" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c", 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" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c", 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" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c" "/opt/rustwide/cargo-home/bin/cargo" "+c60b92c03ca8a3b101eb9d57f51e2232690638c7" "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 syn v3.0.9
[INFO] [stderr]     Checking futures-channel v0.3.34
[INFO] [stderr]     Checking slab v0.4.12
[INFO] [stderr]    Compiling serde v1.0.229
[INFO] [stderr]     Checking aho-corasick v1.1.5
[INFO] [stderr]     Checking bstr v1.13.1
[INFO] [stderr]     Checking lazy_static v1.5.1
[INFO] [stderr]     Checking tokio v1.53.3
[INFO] [stderr]     Checking regex-automata v0.4.18
[INFO] [stderr]    Compiling futures-macro v0.3.34
[INFO] [stderr]    Compiling serde_derive v1.0.229
[INFO] [stderr]     Checking globset v0.4.20
[INFO] [stderr]     Checking futures-util v0.3.34
[INFO] [stderr]     Checking tokio-stream v0.1.19
[INFO] [stderr]     Checking tokio-util v0.6.10
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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` jsonrpc_server_utils
[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" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c", kill_on_drop: false }`
[INFO] [stdout] d099bff4b460fa02f652a308c58638325186ccd140443cd013b5154621382f1c
