[INFO] cloning repository https://github.com/talentprince/HttpServer [INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/talentprince/HttpServer" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftalentprince%2FHttpServer", kill_on_drop: false }` [INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftalentprince%2FHttpServer'... [INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }` [INFO] [stdout] da2247dc23538b2d4448d122886ede0ea9ace943 [INFO] checking talentprince/HttpServer against master#f8f6997469237299c1d60814c7b9828602a1f8e4 for pr-143808 [INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2Ftalentprince%2FHttpServer" "/workspace/builds/worker-2-tc1/source", kill_on_drop: false }` [INFO] [stderr] Cloning into '/workspace/builds/worker-2-tc1/source'... [INFO] [stderr] done. [INFO] started tweaking git repo https://github.com/talentprince/HttpServer [INFO] finished tweaking git repo https://github.com/talentprince/HttpServer [INFO] tweaked toml for git repo https://github.com/talentprince/HttpServer written to /workspace/builds/worker-2-tc1/source/Cargo.toml [INFO] validating manifest of git repo https://github.com/talentprince/HttpServer on toolchain f8f6997469237299c1d60814c7b9828602a1f8e4 [INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+f8f6997469237299c1d60814c7b9828602a1f8e4" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }` [INFO] crate git repo https://github.com/talentprince/HttpServer 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" "+f8f6997469237299c1d60814c7b9828602a1f8e4" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }` [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-e" "CARGO_TARGET_DIR=/opt/rustwide/target" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:cf8efcab8866f2cf4285301c7418e2f4f2a9b088c91ba69c40d5b659f81557f7" "/opt/rustwide/cargo-home/bin/cargo" "+f8f6997469237299c1d60814c7b9828602a1f8e4" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }` [INFO] [stdout] 645642c88cfc7cf87a8a813ea77104bfb66c32a82fe3af9f62dfc0ae43e2639a [INFO] running `Command { std: "docker" "start" "-a" "645642c88cfc7cf87a8a813ea77104bfb66c32a82fe3af9f62dfc0ae43e2639a", kill_on_drop: false }` [INFO] running `Command { std: "docker" "inspect" "645642c88cfc7cf87a8a813ea77104bfb66c32a82fe3af9f62dfc0ae43e2639a", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "645642c88cfc7cf87a8a813ea77104bfb66c32a82fe3af9f62dfc0ae43e2639a", kill_on_drop: false }` [INFO] [stdout] 645642c88cfc7cf87a8a813ea77104bfb66c32a82fe3af9f62dfc0ae43e2639a [INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-2-tc1/source:/opt/rustwide/workdir:ro,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" "-e" "SOURCE_DIR=/opt/rustwide/workdir" "-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" "-e" "CARGO_HOME=/opt/rustwide/cargo-home" "-e" "RUSTUP_HOME=/opt/rustwide/rustup-home" "-w" "/opt/rustwide/workdir" "-m" "1610612736" "--user" "0:0" "--network" "none" "ghcr.io/rust-lang/crates-build-env/linux@sha256:cf8efcab8866f2cf4285301c7418e2f4f2a9b088c91ba69c40d5b659f81557f7" "/opt/rustwide/cargo-home/bin/cargo" "+f8f6997469237299c1d60814c7b9828602a1f8e4" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }` [INFO] [stdout] 49a28b6b6cbe73075d1e5fa46747d93b38a3d0b2c4f803ef56d1ddd8ed8ca28b [INFO] running `Command { std: "docker" "start" "-a" "49a28b6b6cbe73075d1e5fa46747d93b38a3d0b2c4f803ef56d1ddd8ed8ca28b", kill_on_drop: false }` [INFO] [stderr] Checking http_server v0.1.0 (/opt/rustwide/workdir) [INFO] [stdout] warning: unused import: `Value as QueryStringValue` [INFO] [stdout] --> src/http/mod.rs:4:37 [INFO] [stdout] | [INFO] [stdout] 4 | pub use query_string::{QueryString, Value as QueryStringValue}; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused import: `Value as QueryStringValue` [INFO] [stdout] --> src/http/mod.rs:4:37 [INFO] [stdout] | [INFO] [stdout] 4 | pub use query_string::{QueryString, Value as QueryStringValue}; [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^ [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_imports)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `result` [INFO] [stdout] --> src/http/request.rs:37:24 [INFO] [stdout] | [INFO] [stdout] 37 | let (protocal, result) = read_next_word(result).ok_or(ParseError::InvalidReqest)?; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unused variable: `result` [INFO] [stdout] --> src/http/request.rs:37:24 [INFO] [stdout] | [INFO] [stdout] 37 | let (protocal, result) = read_next_word(result).ok_or(ParseError::InvalidReqest)?; [INFO] [stdout] | ^^^^^^ help: if this is intentional, prefix it with an underscore: `_result` [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(unused_variables)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/http/query_string.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] 16 | return self.get(key); [INFO] [stdout] | ------------- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: function cannot return without recursing [INFO] [stdout] --> src/http/query_string.rs:15:5 [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value> { [INFO] [stdout] | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ cannot return without recursing [INFO] [stdout] 16 | return self.get(key); [INFO] [stdout] | ------------- recursive call site [INFO] [stdout] | [INFO] [stdout] = help: a `loop` may express intention better if this is on purpose [INFO] [stdout] = note: `#[warn(unconditional_recursion)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/http_handler.rs:39:17 [INFO] [stdout] | [INFO] [stdout] 39 | _ => Response::new(StatusCode::NotFound, self.read_file("404.html")) [INFO] [stdout] | ^ no value can reach this [INFO] [stdout] | [INFO] [stdout] note: multiple earlier patterns match some of the same values [INFO] [stdout] --> src/http_handler.rs:39:17 [INFO] [stdout] | [INFO] [stdout] 33 | "/" => Response::new(StatusCode::OK, self.read_file("index.html")), [INFO] [stdout] | --- matches some of the same values [INFO] [stdout] 34 | "/welcome" => Response::new(StatusCode::OK, self.read_file("welcome.html")), [INFO] [stdout] | ---------- matches some of the same values [INFO] [stdout] 35 | path => match self.read_file(path) { [INFO] [stdout] | ---- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 39 | _ => Response::new(StatusCode::NotFound, self.read_file("404.html")) [INFO] [stdout] | ^ collectively making this unreachable [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/http/request.rs:26:18 [INFO] [stdout] | [INFO] [stdout] 26 | pub fn query(&self) -> Option<&QueryString> { [INFO] [stdout] | ^^^^^ ------------ [INFO] [stdout] | | || [INFO] [stdout] | | |the lifetimes get resolved as `'_` [INFO] [stdout] | | the lifetimes get resolved as `'_` [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 26 | pub fn query(&self) -> Option<&QueryString<'_>> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/http/query_string.rs:15:16 [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value> { [INFO] [stdout] | ^^^^^ ------ [INFO] [stdout] | | || [INFO] [stdout] | | |the lifetimes get resolved as `'_` [INFO] [stdout] | | the lifetimes get resolved as `'_` [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value<'_>> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: unreachable pattern [INFO] [stdout] --> src/http_handler.rs:39:17 [INFO] [stdout] | [INFO] [stdout] 39 | _ => Response::new(StatusCode::NotFound, self.read_file("404.html")) [INFO] [stdout] | ^ no value can reach this [INFO] [stdout] | [INFO] [stdout] note: multiple earlier patterns match some of the same values [INFO] [stdout] --> src/http_handler.rs:39:17 [INFO] [stdout] | [INFO] [stdout] 33 | "/" => Response::new(StatusCode::OK, self.read_file("index.html")), [INFO] [stdout] | --- matches some of the same values [INFO] [stdout] 34 | "/welcome" => Response::new(StatusCode::OK, self.read_file("welcome.html")), [INFO] [stdout] | ---------- matches some of the same values [INFO] [stdout] 35 | path => match self.read_file(path) { [INFO] [stdout] | ---- matches some of the same values [INFO] [stdout] ... [INFO] [stdout] 39 | _ => Response::new(StatusCode::NotFound, self.read_file("404.html")) [INFO] [stdout] | ^ collectively making this unreachable [INFO] [stdout] = note: `#[warn(unreachable_patterns)]` on by default [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/http/request.rs:26:18 [INFO] [stdout] | [INFO] [stdout] 26 | pub fn query(&self) -> Option<&QueryString> { [INFO] [stdout] | ^^^^^ ------------ [INFO] [stdout] | | || [INFO] [stdout] | | |the lifetimes get resolved as `'_` [INFO] [stdout] | | the lifetimes get resolved as `'_` [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] = note: `#[warn(mismatched_lifetime_syntaxes)]` on by default [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 26 | pub fn query(&self) -> Option<&QueryString<'_>> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stdout] warning: lifetime flowing from input to output with different syntax can be confusing [INFO] [stdout] --> src/http/query_string.rs:15:16 [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value> { [INFO] [stdout] | ^^^^^ ------ [INFO] [stdout] | | || [INFO] [stdout] | | |the lifetimes get resolved as `'_` [INFO] [stdout] | | the lifetimes get resolved as `'_` [INFO] [stdout] | this lifetime flows to the output [INFO] [stdout] | [INFO] [stdout] help: one option is to remove the lifetime for references and use the anonymous lifetime for paths [INFO] [stdout] | [INFO] [stdout] 15 | pub fn get(&self, key: &str) -> Option<&Value<'_>> { [INFO] [stdout] | ++++ [INFO] [stdout] [INFO] [stdout] [INFO] [stderr] Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.13s [INFO] running `Command { std: "docker" "inspect" "49a28b6b6cbe73075d1e5fa46747d93b38a3d0b2c4f803ef56d1ddd8ed8ca28b", kill_on_drop: false }` [INFO] running `Command { std: "docker" "rm" "-f" "49a28b6b6cbe73075d1e5fa46747d93b38a3d0b2c4f803ef56d1ddd8ed8ca28b", kill_on_drop: false }` [INFO] [stdout] 49a28b6b6cbe73075d1e5fa46747d93b38a3d0b2c4f803ef56d1ddd8ed8ca28b