Skip to content

Commit dc91eab

Browse files
committed
add tracing for fetch calls
1 parent b8ee3ce commit dc91eab

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

turbopack/crates/turbo-tasks-fetch/src/lib.rs

+12-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
use anyhow::Result;
66
use turbo_rcstr::RcStr;
7-
use turbo_tasks::{mark_session_dependent, ResolvedVc, Vc};
7+
use turbo_tasks::{duration_span, mark_session_dependent, ResolvedVc, Vc};
88
use turbo_tasks_fs::FileSystemPath;
99
use turbopack_core::issue::{Issue, IssueSeverity, IssueStage, OptionStyledString, StyledString};
1010

@@ -72,11 +72,20 @@ pub async fn fetch(
7272
builder = builder.header("User-Agent", user_agent.as_str());
7373
}
7474

75-
let response = builder.send().await.and_then(|r| r.error_for_status());
75+
let response = {
76+
let _span = duration_span!("fetch request", url = url.as_str());
77+
builder.send().await
78+
}
79+
.and_then(|r| r.error_for_status());
7680
match response {
7781
Ok(response) => {
7882
let status = response.status().as_u16();
79-
let body = response.bytes().await?.to_vec();
83+
84+
let body = {
85+
let _span = duration_span!("fetch response", url = url.as_str());
86+
response.bytes().await?
87+
}
88+
.to_vec();
8089

8190
Ok(Vc::cell(Ok(HttpResponse {
8291
status,

turbopack/crates/turbopack-trace-utils/src/tracing_presets.rs

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,15 @@ pub static TRACING_OVERVIEW_TARGETS: Lazy<Vec<&str>> = Lazy::new(|| {
44
vec![
55
"turbo_tasks=info",
66
"turbo_tasks_fs=info",
7+
"turbo_tasks_fetch=info",
78
"turbopack=info",
89
"turbopack_binding=info",
10+
"turbopack_browser=info",
911
"turbopack_nodejs=info",
1012
"turbopack_cli=info",
1113
"turbopack_cli_utils=info",
1214
"turbopack_core=info",
1315
"turbopack_css=info",
14-
"turbopack_browser=info",
1516
"turbopack_dev_server=info",
1617
"turbopack_ecmascript=info",
1718
"turbopack_ecmascript_hmr_protocol=info",

0 commit comments

Comments
 (0)