fix(registry): explicitly specify a five-second timeout for downloading registry

Author: zwpaper

crates/tabby-common/src/registry.rs

@@ -40,15 +40,27 @@ fn models_json_file(registry: &str) -> PathBuf {
 }
 
 async fn load_remote_registry(registry: &str) -> Result<Vec<ModelInfo>> {
-    let model_info = reqwest::get(format!(
-        "https://raw.githubusercontent.com/{}/registry-tabby/main/models.json",
-        registry
-    ))
-    .await
-    .context("Failed to download")?
-    .json()
-    .await
-    .context("Failed to get JSON")?;
+    // Create a client with custom timeout.
+    // There have been instances where requests did not adhere to the default timeout,
+    // resulting in requests hanging indefinitely.
+    // Therefore, it's necessary to specify a custom timeout.
+    let client = reqwest::Client::builder()
+        .timeout(std::time::Duration::from_secs(5))
+        .build()
+        .context("Failed to build HTTP client")?;
+
+    let model_info = client
+        .get(format!(
+            "https://raw.githubusercontent.com/{}/registry-tabby/main/models.json",
+            registry
+        ))
+        .send()
+        .await
+        .context("Failed to download")?
+        .json()
+        .await
+        .context("Failed to get JSON")?;
+
     let dir = models_dir().join(registry);
     // We don't want to fail if the TabbyML directory already exists,
     // which is exactly, what `create_dir_all` will do, see