René's URL Explorer Experiment

Title: fix: honor HTTP_PROXY / HTTPS_PROXY / NO_PROXY env vars (#35) by XXPermanentXX · Pull Request #51 · modelstudioai/cli · GitHub

Open Graph Title: fix: honor HTTP_PROXY / HTTPS_PROXY / NO_PROXY env vars (#35) by XXPermanentXX · Pull Request #51 · modelstudioai/cli

X Title: fix: honor HTTP_PROXY / HTTPS_PROXY / NO_PROXY env vars (#35) by XXPermanentXX · Pull Request #51 · modelstudioai/cli

Description: Summary Fixes #35 — running bl behind a VPN / corporate proxy failed with ECONNRESET: Client network socket disconnected before secure TLS connection was established. Root cause: Node's built-in fetch (undici) does not read proxy environment variables, so the CLI always connected directly, bypassing the local proxy and getting blocked. Fix: at startup, install an EnvHttpProxyAgent as the global undici dispatcher — but only when HTTP_PROXY / HTTPS_PROXY is actually set. When no proxy variable is configured, the global dispatcher is untouched and behavior is exactly as before. export HTTPS_PROXY=http://127.0.0.1:7890 bl app list # now routes through the proxy Changes packages/cli/src/proxy.ts (new): readProxyEnv() + setupProxyFromEnv(). Lowercase variables take precedence over uppercase (curl convention); NO_PROXY is supported. Values are trimmed and passed explicitly to work around undici reading env vars with ??, where an empty lowercase variable (https_proxy="") masks a configured uppercase one. An invalid proxy URL fails with a clear Invalid proxy configuration: … error (plus hint) instead of a raw stack trace. packages/cli/src/main.ts: call setupProxyFromEnv() at module top level, before any fetch is issued (including update-checker / telemetry). packages/cli/src/error-handler.ts: the ECONNRESET hint now suggests export HTTPS_PROXY=… for VPN / corporate-proxy users. undici added as an explicit dependency (catalog ^8.4.1). Tests (fully offline, no credentials required) tests/proxy.test.ts: readProxyEnv parsing — unset/blank handling, lowercase precedence, empty-string-masking regression, NO_PROXY. tests/e2e/proxy.e2e.test.ts: runs a minimal probe script (setupProxyFromEnv() + a bare fetch()) against a .invalid host through a local CONNECT proxy. It does not drive any CLI command, so it needs no api key / access token — consistent with the existing e2e design. Verifies: with HTTPS_PROXY set, the bare fetch tunnels through the proxy (CONNECT to the target host); empty lowercase https_proxy does not mask HTTPS_PROXY; NO_PROXY match keeps the request direct; with no proxy vars set, the proxy receives zero traffic (no behavior change); an invalid proxy URL produces a clear error message. All proxy tests pass under a simulated CI environment (empty HOME, no credentials). Note Supersedes #50, which had a tangled commit history. This branch is rebuilt cleanly on top of main. The version bump / CHANGELOG for the release are intentionally left out and will go in a separate PR. Also includes two small unrelated cleanups as separate commits: a video-ref r2v e2e prompt tweak, and removing the machine-local .claude/scheduled_tasks.lock from version control.

Open Graph Description: Summary Fixes #35 — running bl behind a VPN / corporate proxy failed with ECONNRESET: Client network socket disconnected before secure TLS connection was established. Root cause: Node's built-i...

X Description: Summary Fixes #35 — running bl behind a VPN / corporate proxy failed with ECONNRESET: Client network socket disconnected before secure TLS connection was established. Root cause: Node's bui...

Opengraph URL: https://github.com/modelstudioai/cli/pull/51

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width