fix(install): match wget's 20s connection tolerance in curl helpers

thomas.szbay · thomas.szbay · commit edb70f89e971 · 2026-07-19T08:05:33.000+08:00
Per review feedback: --connect-timeout covers DNS, TCP, and TLS setup, so
a 10-second cutoff could fail valid slow connection establishment that
the wget sibling (--timeout=20) still permits. Use 20 seconds in both
curl helpers to match the wget connection window.
diff --git a/scripts/install-cli.sh b/scripts/install-cli.sh
@@ -140,7 +140,7 @@ download_file() {
     # Bound connection setup and post-connect stalls without imposing a
     # total download duration.
     curl -fsSL --proto '=https' --tlsv1.2 \
-      --connect-timeout 10 \
+      --connect-timeout 20 \
       --speed-limit 1 --speed-time 30 \
       --retry 3 --retry-delay 1 --retry-connrefused \
       -o "$output" "$url"
diff --git a/scripts/install.sh b/scripts/install.sh
@@ -117,7 +117,7 @@ download_file() {
         # Bound connection setup and post-connect stalls without imposing a
         # total download duration.
         curl -fsSL --proto '=https' --tlsv1.2 \
-            --connect-timeout 10 \
+            --connect-timeout 20 \
             --speed-limit 1 --speed-time 30 \
             --retry 3 --retry-delay 1 --retry-connrefused \
             -o "$output" "$url"
diff --git a/test/scripts/install-cli.test.ts b/test/scripts/install-cli.test.ts
@@ -56,7 +56,7 @@ describe("install-cli.sh", () => {
     `);
 
     expect(result.status).toBe(0);
-    expect(result.stdout).toContain("--connect-timeout 10");
+    expect(result.stdout).toContain("--connect-timeout 20");
     expect(result.stdout).toContain("--speed-limit 1 --speed-time 30");
     expect(result.stdout).toContain("--retry 3 --retry-delay 1 --retry-connrefused");
     expect(result.stdout).toContain("status=28");
diff --git a/test/scripts/install-sh.test.ts b/test/scripts/install-sh.test.ts
@@ -97,7 +97,7 @@ describe("install.sh", () => {
     `);
 
     expect(result.status).toBe(0);
-    expect(result.stdout).toContain("--connect-timeout 10");
+    expect(result.stdout).toContain("--connect-timeout 20");
     expect(result.stdout).toContain("--speed-limit 1 --speed-time 30");
     expect(result.stdout).toContain("--retry 3 --retry-delay 1 --retry-connrefused");
     expect(result.stdout).toContain("status=28");
