From da6dc375d679d302e379214564913aee7ba6f722 Mon Sep 17 00:00:00 2001 From: nsfisis Date: Sun, 28 Jun 2026 18:50:24 +0900 Subject: fix(http): avoid nested tokio runtime panic in download path The CurlDownloader owned a tokio runtime and block_on'd reqwest from its sync tick(), while the repository/installer/downloader sync bridges each created another Runtime and block_on'd async fns that reach that leaf. Driving one Runtime::block_on from within another panics with "Cannot start a runtime from within a runtime", hit by `require` when fetching p2 metadata. Switch CurlDownloader to a blocking reqwest client (its own internal thread, never nested) and replace the per-call Runtime::new().block_on bridges with a no-reactor sync_executor::block_on helper. No awaited future parks on a reactor once the only async I/O is blocking, so the helper can be nested freely. Co-Authored-By: Claude Opus 4.8 (1M context) --- crates/shirabe/src/util.rs | 1 + 1 file changed, 1 insertion(+) (limited to 'crates/shirabe/src/util.rs') diff --git a/crates/shirabe/src/util.rs b/crates/shirabe/src/util.rs index bd51b66..35402d7 100644 --- a/crates/shirabe/src/util.rs +++ b/crates/shirabe/src/util.rs @@ -25,6 +25,7 @@ pub mod remote_filesystem; pub mod silencer; pub mod stream_context_factory; pub mod svn; +pub mod sync_executor; pub mod sync_helper; pub mod tar; pub mod tls_helper; -- cgit v1.3.1