From 7b33623b1d70803b43e511a58666a73dd0b2ed67 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bartek=20Iwa=C5=84czuk?= Date: Thu, 18 Jul 2024 00:37:31 +0100 Subject: Reland "refactor(fetch): reimplement fetch with hyper instead of reqwest" (#24593) Originally landed in https://github.com/denoland/deno/commit/f6fd6619e708a515831f707438368d81b0c9aa56. Reverted in https://github.com/denoland/deno/pull/24574. This reland contains a fix that sends "Accept: */*" header for calls made from "FileFetcher". Absence of this header made downloading source code from JSR broken. This is tested by ensuring this header is present in the test server that servers JSR packages. --------- Co-authored-by: Sean McArthur --- tests/util/server/src/servers/jsr_registry.rs | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'tests/util') diff --git a/tests/util/server/src/servers/jsr_registry.rs b/tests/util/server/src/servers/jsr_registry.rs index ce1a34ac9..affd2c30e 100644 --- a/tests/util/server/src/servers/jsr_registry.rs +++ b/tests/util/server/src/servers/jsr_registry.rs @@ -139,6 +139,17 @@ async fn registry_server_handler( return Ok(res); } + let accept_header = req + .headers() + .get("accept") + .and_then(|s| s.to_str().ok()) + .unwrap_or_default(); + if accept_header != "*/*" { + let res = Response::builder() + .status(StatusCode::BAD_REQUEST) + .body(UnsyncBoxBody::new(Empty::new()))?; + return Ok(res); + } // serve the registry package files let mut file_path = tests_path().join("registry").join("jsr").to_path_buf(); file_path.push( -- cgit v1.2.3