Skip to content

Commit 955f0cc

Browse files
authored
Merge pull request #905 from DeusData/fix/smoke-windows-curl-diag
fix(smoke): --noproxy for local download + surface curl errors (arm64 12a)
2 parents 71e9b3b + 2a664aa commit 955f0cc

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

scripts/smoke-test.sh

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1594,12 +1594,18 @@ DL_ARCHIVE_UI="codebase-memory-mcp-ui-${DL_OS}-${DL_ARCH}.${DL_EXT}"
15941594

15951595
# 12a: curl download (try standard, then UI variant)
15961596
echo "--- Phase 12a: curl download ---"
1597-
if ! curl -fSL -o "$DL_DIR/$DL_ARCHIVE" "$SMOKE_DOWNLOAD_URL/$DL_ARCHIVE" 2>/dev/null; then
1597+
# --noproxy '*': never route the local test server through a proxy — a proxy env
1598+
# var present on some runners (notably windows-11-arm) made curl fail to reach
1599+
# 127.0.0.1 while the app's own downloader (WinHTTP) bypassed it. On failure,
1600+
# surface curl's stderr instead of swallowing it so the reason is visible.
1601+
if ! curl -fSL --noproxy '*' -o "$DL_DIR/$DL_ARCHIVE" "$SMOKE_DOWNLOAD_URL/$DL_ARCHIVE" 2>/tmp/cbm-curl12a.err; then
15981602
# Try UI variant
1599-
if curl -fSL -o "$DL_DIR/$DL_ARCHIVE_UI" "$SMOKE_DOWNLOAD_URL/$DL_ARCHIVE_UI" 2>/dev/null; then
1603+
if curl -fSL --noproxy '*' -o "$DL_DIR/$DL_ARCHIVE_UI" "$SMOKE_DOWNLOAD_URL/$DL_ARCHIVE_UI" 2>>/tmp/cbm-curl12a.err; then
16001604
DL_ARCHIVE="$DL_ARCHIVE_UI"
16011605
else
16021606
echo "FAIL 12a: curl download failed (tried standard and ui variants)"
1607+
echo "--- curl stderr (url: $SMOKE_DOWNLOAD_URL/$DL_ARCHIVE) ---"
1608+
cat /tmp/cbm-curl12a.err 2>/dev/null || true
16031609
exit 1
16041610
fi
16051611
fi

0 commit comments

Comments
 (0)