[INFO] cloning repository https://github.com/KaloyanYosifov/basic-http-server-rust
[INFO] running `Command { std: "git" "-c" "credential.helper=" "-c" "credential.helper=/workspace/cargo-home/bin/git-credential-null" "clone" "--bare" "https://github.com/KaloyanYosifov/basic-http-server-rust" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKaloyanYosifov%2Fbasic-http-server-rust", kill_on_drop: false }`
[INFO] [stderr] Cloning into bare repository '/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKaloyanYosifov%2Fbasic-http-server-rust'...
[INFO] running `Command { std: "git" "rev-parse" "HEAD", kill_on_drop: false }`
[INFO] [stdout] 84dd8cbb7e7e065a2e366978c1c9725c3c9add48
[INFO] checking KaloyanYosifov/basic-http-server-rust/84dd8cbb7e7e065a2e366978c1c9725c3c9add48 against master#5518eaa946291f00471af8b254b2a1715f234882 for pr-120393
[INFO] running `Command { std: "git" "clone" "/workspace/cache/git-repos/https%3A%2F%2Fgithub.com%2FKaloyanYosifov%2Fbasic-http-server-rust" "/workspace/builds/worker-4-tc1/source", kill_on_drop: false }`
[INFO] [stderr] Cloning into '/workspace/builds/worker-4-tc1/source'...
[INFO] [stderr] done.
[INFO] validating manifest of git repo https://github.com/KaloyanYosifov/basic-http-server-rust on toolchain 5518eaa946291f00471af8b254b2a1715f234882
[INFO] running `Command { std: CARGO_HOME="/workspace/cargo-home" RUSTUP_HOME="/workspace/rustup-home" "/workspace/cargo-home/bin/cargo" "+5518eaa946291f00471af8b254b2a1715f234882" "metadata" "--manifest-path" "Cargo.toml" "--no-deps", kill_on_drop: false }`
[INFO] started tweaking git repo https://github.com/KaloyanYosifov/basic-http-server-rust
[INFO] finished tweaking git repo https://github.com/KaloyanYosifov/basic-http-server-rust
[INFO] tweaked toml for git repo https://github.com/KaloyanYosifov/basic-http-server-rust written to /workspace/builds/worker-4-tc1/source/Cargo.toml
[INFO] crate git repo https://github.com/KaloyanYosifov/basic-http-server-rust 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" "+5518eaa946291f00471af8b254b2a1715f234882" "fetch" "--manifest-path" "Cargo.toml", kill_on_drop: false }`
[INFO] [stderr]  Downloading crates ...
[INFO] [stderr]   Downloaded serde_json v1.0.65
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:2788e3201cd34a07e3172128adcd8b3090168a8e3bcc40d7c032b9dda1df7d1c" "/opt/rustwide/cargo-home/bin/cargo" "+5518eaa946291f00471af8b254b2a1715f234882" "metadata" "--no-deps" "--format-version=1", kill_on_drop: false }`
[INFO] [stdout] a05bb97b28ecdc0168e1b923babad619c1efb8650bce930ef83395b2bd10820e
[INFO] running `Command { std: "docker" "start" "-a" "a05bb97b28ecdc0168e1b923babad619c1efb8650bce930ef83395b2bd10820e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "inspect" "a05bb97b28ecdc0168e1b923babad619c1efb8650bce930ef83395b2bd10820e", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "a05bb97b28ecdc0168e1b923babad619c1efb8650bce930ef83395b2bd10820e", kill_on_drop: false }`
[INFO] [stdout] a05bb97b28ecdc0168e1b923babad619c1efb8650bce930ef83395b2bd10820e
[INFO] running `Command { std: "docker" "create" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-tc1/target:/opt/rustwide/target:rw,Z" "-v" "/var/lib/crater-agent-workspace/builds/worker-4-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:2788e3201cd34a07e3172128adcd8b3090168a8e3bcc40d7c032b9dda1df7d1c" "/opt/rustwide/cargo-home/bin/cargo" "+5518eaa946291f00471af8b254b2a1715f234882" "check" "--frozen" "--all" "--all-targets" "--message-format=json", kill_on_drop: false }`
[INFO] [stdout] c63475e887ee03de1b719522209235c1a0e70729f7ac58ba684c787326b9c96a
[INFO] running `Command { std: "docker" "start" "-a" "c63475e887ee03de1b719522209235c1a0e70729f7ac58ba684c787326b9c96a", kill_on_drop: false }`
[INFO] [stderr]    Compiling proc-macro2 v1.0.29
[INFO] [stderr]    Compiling syn v1.0.76
[INFO] [stderr]     Checking regex-syntax v0.6.25
[INFO] [stderr]    Compiling serde v1.0.126
[INFO] [stderr]     Checking aho-corasick v0.7.18
[INFO] [stderr]     Checking unicode-xid v0.2.2
[INFO] [stderr]    Compiling lexical-core v0.7.6
[INFO] [stderr]    Compiling nom v5.1.2
[INFO] [stderr]    Compiling serde_json v1.0.65
[INFO] [stderr]     Checking static_assertions v1.1.0
[INFO] [stderr]     Checking arrayvec v0.5.2
[INFO] [stderr]     Checking num-traits v0.1.43
[INFO] [stderr]     Checking serde v0.8.23
[INFO] [stderr]     Checking linked-hash-map v0.5.4
[INFO] [stderr]     Checking itoa v0.4.7
[INFO] [stderr]     Checking quote v1.0.9
[INFO] [stderr]     Checking yaml-rust v0.4.5
[INFO] [stderr]     Checking rust-ini v0.13.0
[INFO] [stderr]     Checking regex v1.5.4
[INFO] [stderr]     Checking http v0.1.0 (/opt/rustwide/workdir/http)
[INFO] [stderr]     Checking serde-hjson v0.9.1
[INFO] [stdout] warning: unused imports: `Response`, `StatusCode`
[INFO] [stdout]  --> http/src/lib.rs:7:23
[INFO] [stdout]   |
[INFO] [stdout] 7 | use crate::response::{Response, StatusCode};
[INFO] [stdout]   |                       ^^^^^^^^  ^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::panic::resume_unwind`
[INFO] [stdout]  --> http/src/lib.rs:9:5
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::panic::resume_unwind;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused imports: `Response`, `StatusCode`
[INFO] [stdout]  --> http/src/lib.rs:7:23
[INFO] [stdout]   |
[INFO] [stdout] 7 | use crate::response::{Response, StatusCode};
[INFO] [stdout]   |                       ^^^^^^^^  ^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `std::panic::resume_unwind`
[INFO] [stdout]  --> http/src/lib.rs:9:5
[INFO] [stdout]   |
[INFO] [stdout] 9 | use std::panic::resume_unwind;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Write`
[INFO] [stdout]  --> http/src/lib.rs:2:21
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::io::{Read, Write};
[INFO] [stdout]   |                     ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `route`
[INFO] [stdout]   --> http/src/route.rs:54:13
[INFO] [stdout]    |
[INFO] [stdout] 54 |         let route = Route::new("/");
[INFO] [stdout]    |             ^^^^^ help: if this is intentional, prefix it with an underscore: `_route`
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_variables)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unreachable pattern
[INFO] [stdout]   --> http/src/route.rs:73:21
[INFO] [stdout]    |
[INFO] [stdout] 73 |                     _ => panic!("This should have failed!")
[INFO] [stdout]    |                     ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unreachable_patterns)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Write`
[INFO] [stdout]  --> http/src/lib.rs:2:21
[INFO] [stdout]   |
[INFO] [stdout] 2 | use std::io::{Read, Write};
[INFO] [stdout]   |                     ^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unreachable pattern
[INFO] [stdout]   --> http/src/response.rs:18:13
[INFO] [stdout]    |
[INFO] [stdout] 18 |             _ => "Unknown"
[INFO] [stdout]    |             ^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unreachable_patterns)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `request`
[INFO] [stdout]  --> http/src/request_handler.rs:6:28
[INFO] [stdout]   |
[INFO] [stdout] 6 |     fn handle_error(&self, request: &RequestError) -> Response {
[INFO] [stdout]   |                            ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_request`
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_variables)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0170]: pattern binding `GET` is named the same as one of the variants of the type `Method`
[INFO] [stdout]    --> http/src/request.rs:127:42
[INFO] [stdout]     |
[INFO] [stdout] 127 |         assert!(matches!(request.method, GET));
[INFO] [stdout]     |                                          ^^^ help: to match on the variant, qualify the path: `Method::GET`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[deny(bindings_with_variant_name)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `GET`
[INFO] [stdout]    --> http/src/request.rs:127:42
[INFO] [stdout]     |
[INFO] [stdout] 127 |         assert!(matches!(request.method, GET));
[INFO] [stdout]     |                                          ^^^ help: if this is intentional, prefix it with an underscore: `_GET`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0170]: pattern binding `POST` is named the same as one of the variants of the type `Method`
[INFO] [stdout]    --> http/src/request.rs:137:42
[INFO] [stdout]     |
[INFO] [stdout] 137 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: to match on the variant, qualify the path: `Method::POST`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `POST`
[INFO] [stdout]    --> http/src/request.rs:137:42
[INFO] [stdout]     |
[INFO] [stdout] 137 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: if this is intentional, prefix it with an underscore: `_POST`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0170]: pattern binding `POST` is named the same as one of the variants of the type `Method`
[INFO] [stdout]    --> http/src/request.rs:147:42
[INFO] [stdout]     |
[INFO] [stdout] 147 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: to match on the variant, qualify the path: `Method::POST`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `POST`
[INFO] [stdout]    --> http/src/request.rs:147:42
[INFO] [stdout]     |
[INFO] [stdout] 147 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: if this is intentional, prefix it with an underscore: `_POST`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unreachable pattern
[INFO] [stdout]   --> http/src/response.rs:18:13
[INFO] [stdout]    |
[INFO] [stdout] 18 |             _ => "Unknown"
[INFO] [stdout]    |             ^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> http/src/response.rs:48:9
[INFO] [stdout]    |
[INFO] [stdout] 48 | /         write!(
[INFO] [stdout] 49 | |             writer,
[INFO] [stdout] 50 | |             "HTTP/1.1 {status} {status_message}\r\nContent-Length: {content_length}\r\n\r\n{content}",
[INFO] [stdout] 51 | |             status = self.status as i32,
[INFO] [stdout] ...  |
[INFO] [stdout] 54 | |             content = self.content
[INFO] [stdout] 55 | |         );
[INFO] [stdout]    | |_________^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]    = note: `#[warn(unused_must_use)]` on by default
[INFO] [stdout]    = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused variable: `request`
[INFO] [stdout]  --> http/src/request_handler.rs:6:28
[INFO] [stdout]   |
[INFO] [stdout] 6 |     fn handle_error(&self, request: &RequestError) -> Response {
[INFO] [stdout]   |                            ^^^^^^^ help: if this is intentional, prefix it with an underscore: `_request`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: 6 warnings emitted
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr]     Checking core v0.1.0 (/opt/rustwide/workdir/core)
[INFO] [stdout] warning: variable `GET` should have a snake case name
[INFO] [stdout]    --> http/src/request.rs:127:42
[INFO] [stdout]     |
[INFO] [stdout] 127 |         assert!(matches!(request.method, GET));
[INFO] [stdout]     |                                          ^^^ help: convert the identifier to snake case: `get`
[INFO] [stdout]     |
[INFO] [stdout]     = note: `#[warn(non_snake_case)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable `POST` should have a snake case name
[INFO] [stdout]    --> http/src/request.rs:137:42
[INFO] [stdout]     |
[INFO] [stdout] 137 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: convert the identifier to snake case: `post`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: variable `POST` should have a snake case name
[INFO] [stdout]    --> http/src/request.rs:147:42
[INFO] [stdout]     |
[INFO] [stdout] 147 |         assert!(matches!(request.method, POST));
[INFO] [stdout]     |                                          ^^^^ help: convert the identifier to snake case: `post`
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused `Result` that must be used
[INFO] [stdout]   --> http/src/response.rs:48:9
[INFO] [stdout]    |
[INFO] [stdout] 48 | /         write!(
[INFO] [stdout] 49 | |             writer,
[INFO] [stdout] 50 | |             "HTTP/1.1 {status} {status_message}\r\nContent-Length: {content_length}\r\n\r\n{content}",
[INFO] [stdout] 51 | |             status = self.status as i32,
[INFO] [stdout] ...  |
[INFO] [stdout] 54 | |             content = self.content
[INFO] [stdout] 55 | |         );
[INFO] [stdout]    | |_________^
[INFO] [stdout]    |
[INFO] [stdout]    = note: this `Result` may be an `Err` variant, which should be handled
[INFO] [stdout]    = note: `#[warn(unused_must_use)]` on by default
[INFO] [stdout]    = note: this warning originates in the macro `write` (in Nightly builds, run with -Z macro-backtrace for more info)
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: aborting due to 3 previous errors; 14 warnings emitted
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] For more information about this error, try `rustc --explain E0170`.
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `http` (lib test) due to 4 previous errors; 14 warnings emitted
[INFO] [stderr] warning: build failed, waiting for other jobs to finish...
[INFO] [stdout] warning: unused import: `crate::route::Route`
[INFO] [stdout]  --> core/src/controller/request_controller.rs:5:5
[INFO] [stdout]   |
[INFO] [stdout] 5 | use crate::route::Route;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout]   |
[INFO] [stdout]   = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `Response`
[INFO] [stdout]   --> core/src/route.rs:40:26
[INFO] [stdout]    |
[INFO] [stdout] 40 |     use http::response::{Response, StatusCode};
[INFO] [stdout]    |                          ^^^^^^^^
[INFO] [stdout]    |
[INFO] [stdout]    = note: `#[warn(unused_imports)]` on by default
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::route::Route`
[INFO] [stdout]  --> core/src/controller/request_controller.rs:5:5
[INFO] [stdout]   |
[INFO] [stdout] 5 | use crate::route::Route;
[INFO] [stdout]   |     ^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error[E0061]: this method takes 2 arguments but 1 argument was supplied
[INFO] [stdout]   --> core/src/route.rs:57:41
[INFO] [stdout]    |
[INFO] [stdout] 57 |         let response = route.controller.handle(&request);
[INFO] [stdout]    |                                         ^^^^^^---------- an argument of type `&RequestController<'_>` is missing
[INFO] [stdout]    |
[INFO] [stdout] note: method defined here
[INFO] [stdout]   --> core/src/controller/mod.rs:12:8
[INFO] [stdout]    |
[INFO] [stdout] 12 |     fn handle(&self, request: &Request, request_controller: &RequestController) -> Response;
[INFO] [stdout]    |        ^^^^^^
[INFO] [stdout] help: provide the argument
[INFO] [stdout]    |
[INFO] [stdout] 57 |         let response = route.controller.handle(&request, /* &RequestController<'_> */);
[INFO] [stdout]    |                                               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] warning: unused import: `crate::controller::Controller`
[INFO] [stdout]   --> core/src/route.rs:38:9
[INFO] [stdout]    |
[INFO] [stdout] 38 |     use crate::controller::Controller;
[INFO] [stdout]    |         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] error: aborting due to 1 previous error; 3 warnings emitted
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stdout] For more information about this error, try `rustc --explain E0061`.
[INFO] [stdout] 
[INFO] [stdout] warning: 1 warning emitted
[INFO] [stdout] 
[INFO] [stdout] 
[INFO] [stderr] error: could not compile `core` (lib test) due to 2 previous errors; 3 warnings emitted
[INFO] running `Command { std: "docker" "inspect" "c63475e887ee03de1b719522209235c1a0e70729f7ac58ba684c787326b9c96a", kill_on_drop: false }`
[INFO] running `Command { std: "docker" "rm" "-f" "c63475e887ee03de1b719522209235c1a0e70729f7ac58ba684c787326b9c96a", kill_on_drop: false }`
[INFO] [stdout] c63475e887ee03de1b719522209235c1a0e70729f7ac58ba684c787326b9c96a
