Repository navigation
fix(cli): apply configured API request timeout - #1621
John-David Dalton (jdalton) wants to merge 1 commit into
Conversation
83a3038 to
aa8b7f7
Compare
c189096 to
21db608
Compare
|
Cursor (@cursor) review |
PR SummaryMedium Risk Overview
Adds a unit test for Reviewed by Cursor Bugbot for commit 4488885. Configure here. |
fafe2a0 to
4488885
Compare
|
Cursor (@cursor) review |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 4488885. Configure here.
364fa79 to
05e22f4
Compare
Honor configured timeout for direct CLI API calls and carry it across redirects. Assisted-by: Codex:GPT-6
05e22f4 to
c63e20b
Compare

Make direct Socket API requests honor
SOCKET_CLI_API_TIMEOUT, including redirects and stalled response bodies.Add HTTPS integration tests for configured and unset timeouts, redirects, and response-body stalls. Generate a disposable localhost certificate in
os.tmpdir()that expires after three minutes; no certificate or private key is committed.