[INFO] cloning repository https://github.com/EricLemieux/Rust-Web-Server
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/EricLemieux/Rust-Web-Server" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FEricLemieux%2FRust-Web-Server", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FEricLemieux%2FRust-Web-Server'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 68aeb415c057a4b2b7f97dd10eeef90d71f84c3c
[INFO] checking EricLemieux/Rust-Web-Server against try#d8be25549d5418405531341b4be6d8538fc3c42e for pr-156160
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FEricLemieux%2FRust-Web-Server" "/workspace/builds/worker-6-tc2/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-6-tc2/source'...
[INFO] [stderr] done.
[INFO] started tweaking git repo https://github.com/EricLemieux/Rust-Web-Server
[INFO] finished tweaking git repo https://github.com/EricLemieux/Rust-Web-Server
[INFO] tweaked toml for git repo https://github.com/EricLemieux/Rust-Web-Server written to /workspace/builds/worker-6-tc2/source/Cargo.toml
[INFO] validating manifest of git repo https://github.com/EricLemieux/Rust-Web-Server on toolchain d8be25549d5418405531341b4be6d8538fc3c42e
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+d8be25549d5418405531341b4be6d8538fc3c42e" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] crate git repo https://github.com/EricLemieux/Rust-Web-Server 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" "+d8be25549d5418405531341b4be6d8538fc3c42e" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]     Blocking waiting for file lock on package cache
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded strum v0.21.0
[INFO] [stderr]   Downloaded strum_macros v0.21.1
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-tc2/source:/opt/rustwide/workdir:ro,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-6-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:3a6becf2bc8dde7f3fa57ede90e4f284e72d296796fc446bbb1e2c7cc0530151" "sleep" "infinity", kill_on_drop: false }`
[INFO] [stdout] 6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf
[INFO] running `Command { std: "docker" "start" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf", 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" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf" "/opt/rustwide/cargo-home/bin/cargo" "+d8be25549d5418405531341b4be6d8538fc3c42e" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf", 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" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf" "/opt/rustwide/cargo-home/bin/cargo" "+d8be25549d5418405531341b4be6d8538fc3c42e" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stderr]    Compiling syn v1.0.75
[INFO] [stderr]    Compiling unicode-segmentation v1.8.0
[INFO] [stderr]     Checking strum v0.21.0
[INFO] [stderr]    Compiling heck v0.3.3
[INFO] [stderr]    Compiling strum_macros v0.21.1
[INFO] [stderr]     Checking rust-web-server v0.1.0 (/opt/rustwide/workdir)
[INFO] [stdout] warning: unused import: `std::borrow::Borrow`
[INFO] [stdout]  --> src/http.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::borrow::Borrow;
[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: `std::borrow::Borrow`
[INFO] [stdout]  --> src/http.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::borrow::Borrow;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `headers` is never read
[INFO] [stdout]   --> src/http.rs:30:5
[INFO] [stdout]    |
[INFO] [stdout] 25 | pub struct HttpRequest {
[INFO] [stdout]    |            ----------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 30 |     headers: HashMap<String, String>,
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `HttpRequest` 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 `headers` is never read
[INFO] [stdout]    --> src/http.rs:104:5
[INFO] [stdout]     |
[INFO] [stdout] 101 | pub struct HttpResponse {
[INFO] [stdout]     |            ------------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 104 |     headers: HashMap<String, String>,
[INFO] [stdout]     |     ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `HttpResponse` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.borrow()` on a reference in this situation does nothing
[INFO] [stdout]   --> src/http/http_status.rs:50:36
[INFO] [stdout]    |
[INFO] [stdout] 50 |                 return Result::Ok(i.borrow());
[INFO] [stdout]    |                                    ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: the type `HttpStatus` does not implement `Borrow`, so calling `borrow` on `&HttpStatus` copies the reference, which does not do anything and can be removed
[INFO] [stdout]    = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]    |
[INFO] [stdout] 50 -                 return Result::Ok(i.borrow());
[INFO] [stdout] 50 +                 return Result::Ok(i);
[INFO] [stdout]    |
[INFO] [stdout] help: if you meant to clone `HttpStatus`, implement `Clone` for it
[INFO] [stdout]    |
[INFO] [stdout]  7 + #[derive(Clone)]
[INFO] [stdout]  8 | pub struct HttpStatus {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `headers` is never read
[INFO] [stdout]   --> src/http.rs:30:5
[INFO] [stdout]    |
[INFO] [stdout] 25 | pub struct HttpRequest {
[INFO] [stdout]    |            ----------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 30 |     headers: HashMap<String, String>,
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `HttpRequest` 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 `headers` is never read
[INFO] [stdout]    --> src/http.rs:104:5
[INFO] [stdout]     |
[INFO] [stdout] 101 | pub struct HttpResponse {
[INFO] [stdout]     |            ------------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 104 |     headers: HashMap<String, String>,
[INFO] [stdout]     |     ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `HttpResponse` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.borrow()` on a reference in this situation does nothing
[INFO] [stdout]   --> src/http/http_status.rs:50:36
[INFO] [stdout]    |
[INFO] [stdout] 50 |                 return Result::Ok(i.borrow());
[INFO] [stdout]    |                                    ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: the type `http_status::HttpStatus` does not implement `Borrow`, so calling `borrow` on `&http_status::HttpStatus` copies the reference, which does not do anything and can be removed
[INFO] [stdout]    = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]    |
[INFO] [stdout] 50 -                 return Result::Ok(i.borrow());
[INFO] [stdout] 50 +                 return Result::Ok(i);
[INFO] [stdout]    |
[INFO] [stdout] help: if you meant to clone `http_status::HttpStatus`, implement `Clone` for it
[INFO] [stdout]    |
[INFO] [stdout]  7 + #[derive(Clone)]
[INFO] [stdout]  8 | pub struct HttpStatus {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::borrow::Borrow`
[INFO] [stdout]  --> src/http.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::borrow::Borrow;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: enum `HttpMethod` is never used
[INFO] [stdout]   --> src/http.rs:10:10
[INFO] [stdout]    |
[INFO] [stdout] 10 | pub enum HttpMethod {
[INFO] [stdout]    |          ^^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(dead_code)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: enum `HttpVersion` is never used
[INFO] [stdout]   --> src/http.rs:19:10
[INFO] [stdout]    |
[INFO] [stdout] 19 | pub enum HttpVersion {
[INFO] [stdout]    |          ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `HttpRequest` is never constructed
[INFO] [stdout]   --> src/http.rs:25:12
[INFO] [stdout]    |
[INFO] [stdout] 25 | pub struct HttpRequest {
[INFO] [stdout]    |            ^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `HttpResponse` is never constructed
[INFO] [stdout]    --> src/http.rs:101:12
[INFO] [stdout]     |
[INFO] [stdout] 101 | pub struct HttpResponse {
[INFO] [stdout]     |            ^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `new`, `status`, and `body` are never used
[INFO] [stdout]    --> src/http.rs:109:12
[INFO] [stdout]     |
[INFO] [stdout] 108 | impl HttpResponse {
[INFO] [stdout]     | ----------------- associated items in this implementation
[INFO] [stdout] 109 |     pub fn new() -> Self {
[INFO] [stdout]     |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 118 |     pub fn status(&mut self, status: HttpStatus) {
[INFO] [stdout]     |            ^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 122 |     pub fn body(&mut self, body: String) {
[INFO] [stdout]     |            ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: struct `HttpStatus` is never constructed
[INFO] [stdout]  --> src/http/http_status.rs:7:12
[INFO] [stdout]   |
[INFO] [stdout] 7 | pub struct HttpStatus {
[INFO] [stdout]   |            ^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `OK`, `CREATED`, `ACCEPTED`, `BAD_REQUEST`, `AVAILABLE`, and `from` are never used
[INFO] [stdout]   --> src/http/http_status.rs:16:15
[INFO] [stdout]    |
[INFO] [stdout] 12 | impl HttpStatus {
[INFO] [stdout]    | --------------- associated items in this implementation
[INFO] [stdout] ...
[INFO] [stdout] 16 |     pub const OK: HttpStatus = HttpStatus {
[INFO] [stdout]    |               ^^
[INFO] [stdout] ...
[INFO] [stdout] 20 |     pub const CREATED: HttpStatus = HttpStatus {
[INFO] [stdout]    |               ^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 24 |     pub const ACCEPTED: HttpStatus = HttpStatus {
[INFO] [stdout]    |               ^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 32 |     pub const BAD_REQUEST: HttpStatus = HttpStatus {
[INFO] [stdout]    |               ^^^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 40 |     const AVAILABLE: [&'static HttpStatus; 4] = [
[INFO] [stdout]    |           ^^^^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 47 |     pub fn from(code: u16) -> Result<&'static Self, String> {
[INFO] [stdout]    |            ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.borrow()` on a reference in this situation does nothing
[INFO] [stdout]   --> src/http/http_status.rs:50:36
[INFO] [stdout]    |
[INFO] [stdout] 50 |                 return Result::Ok(i.borrow());
[INFO] [stdout]    |                                    ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: the type `http::http_status::HttpStatus` does not implement `Borrow`, so calling `borrow` on `&http::http_status::HttpStatus` copies the reference, which does not do anything and can be removed
[INFO] [stdout]    = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]    |
[INFO] [stdout] 50 -                 return Result::Ok(i.borrow());
[INFO] [stdout] 50 +                 return Result::Ok(i);
[INFO] [stdout]    |
[INFO] [stdout] help: if you meant to clone `http::http_status::HttpStatus`, implement `Clone` for it
[INFO] [stdout]    |
[INFO] [stdout]  7 + #[derive(Clone)]
[INFO] [stdout]  8 | pub struct HttpStatus {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::borrow::Borrow`
[INFO] [stdout]  --> src/http.rs:2:5
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::borrow::Borrow;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` (part of `#[warn(unused)]`) on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: field `headers` is never read
[INFO] [stdout]   --> src/http.rs:30:5
[INFO] [stdout]    |
[INFO] [stdout] 25 | pub struct HttpRequest {
[INFO] [stdout]    |            ----------- field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 30 |     headers: HashMap<String, String>,
[INFO] [stdout]    |     ^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `HttpRequest` 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 `headers` is never read
[INFO] [stdout]    --> src/http.rs:104:5
[INFO] [stdout]     |
[INFO] [stdout] 101 | pub struct HttpResponse {
[INFO] [stdout]     |            ------------ field in this struct
[INFO] [stdout] ...
[INFO] [stdout] 104 |     headers: HashMap<String, String>,
[INFO] [stdout]     |     ^^^^^^^
[INFO] [stdout]     |
[INFO] [stdout]     = note: `HttpResponse` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: associated items `new`, `status`, and `body` are never used
[INFO] [stdout]    --> src/http.rs:109:12
[INFO] [stdout]     |
[INFO] [stdout] 108 | impl HttpResponse {
[INFO] [stdout]     | ----------------- associated items in this implementation
[INFO] [stdout] 109 |     pub fn new() -> Self {
[INFO] [stdout]     |            ^^^
[INFO] [stdout] ...
[INFO] [stdout] 118 |     pub fn status(&mut self, status: HttpStatus) {
[INFO] [stdout]     |            ^^^^^^
[INFO] [stdout] ...
[INFO] [stdout] 122 |     pub fn body(&mut self, body: String) {
[INFO] [stdout]     |            ^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: call to `.borrow()` on a reference in this situation does nothing
[INFO] [stdout]   --> src/http/http_status.rs:50:36
[INFO] [stdout]    |
[INFO] [stdout] 50 |                 return Result::Ok(i.borrow());
[INFO] [stdout]    |                                    ^^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: the type `http::http_status::HttpStatus` does not implement `Borrow`, so calling `borrow` on `&http::http_status::HttpStatus` copies the reference, which does not do anything and can be removed
[INFO] [stdout]    = note: `#[warn(noop_method_call)]` on by default
[INFO] [stdout] help: remove this redundant call
[INFO] [stdout]    |
[INFO] [stdout] 50 -                 return Result::Ok(i.borrow());
[INFO] [stdout] 50 +                 return Result::Ok(i);
[INFO] [stdout]    |
[INFO] [stdout] help: if you meant to clone `http::http_status::HttpStatus`, implement `Clone` for it
[INFO] [stdout]    |
[INFO] [stdout]  7 + #[derive(Clone)]
[INFO] [stdout]  8 | pub struct HttpStatus {
[INFO] [stdout]    |
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Finished `dev` profile [unoptimized + debuginfo] target(s) in 5.14s
[INFO] running `Command { std: "docker" "inspect" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf", kill_on_drop: false }`
[INFO] [stdout] 6e5a1cf8926990ab24c1e02b52bba539537619442b2613ae2e0877f4a3559aaf
