mirror of
https://github.com/vhaudiquet/ecrou.git
synced 2026-11-12 22:22:44 +00:00
fix(download): bound the metadata clients like the transfers
The freshness digest check and the distro checksum fetch kept the total 30 s request timeout the transfer clients just lost. They move to connect + read windows, matching the one policy.
This commit is contained in:
1 parent
47c9208aab
commit
bb61be4625
1 file changed
+6
-2
@@ -96,8 +96,10 @@ async fn fetch_oci_digest_async(
|
|||||||
repository: &str,
|
repository: &str,
|
||||||
tag: &str,
|
tag: &str,
|
||||||
) -> anyhow::Result<String> {
|
) -> anyhow::Result<String> {
|
||||||
|
// Metadata call: a small body, so the read window alone bounds it.
|
||||||
let client = Client::builder()
|
let client = Client::builder()
|
||||||
.timeout(Duration::from_secs(30))
|
.connect_timeout(Duration::from_secs(30))
|
||||||
|
.read_timeout(Duration::from_secs(30))
|
||||||
.build()
|
.build()
|
||||||
.context("Failed to create HTTP client")?;
|
.context("Failed to create HTTP client")?;
|
||||||
|
|
||||||
@@ -215,8 +217,10 @@ async fn fetch_published_checksum(distro: &Distro, tarball_url: &str) -> anyhow:
|
|||||||
return Err(anyhow!("no checksum artifact for {}", tarball_url));
|
return Err(anyhow!("no checksum artifact for {}", tarball_url));
|
||||||
};
|
};
|
||||||
|
|
||||||
|
// Metadata call: a small body, so the read window alone bounds it.
|
||||||
let client = Client::builder()
|
let client = Client::builder()
|
||||||
.timeout(Duration::from_secs(30))
|
.connect_timeout(Duration::from_secs(30))
|
||||||
|
.read_timeout(Duration::from_secs(30))
|
||||||
.build()
|
.build()
|
||||||
.context("Failed to create HTTP client")?;
|
.context("Failed to create HTTP client")?;
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user