Skip to content

Commit 9267c2c

Browse files
committed
Revert "🤖 fix: add timeout to SSH background spawn to protect against connection hangs"
This reverts commit 4993197.
1 parent 4993197 commit 9267c2c

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

‎src/node/runtime/SSHRuntime.ts‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -376,11 +376,11 @@ export class SSHRuntime implements Runtime {
376376
});
377377

378378
try {
379-
// The spawn command backgrounds the process and returns immediately with the PID.
380-
// Timeout protects against SSH connection hangs (network issues), not the spawned process.
379+
// No timeout - the spawn command backgrounds the process and returns immediately,
380+
// but if wrapped in `timeout`, it would wait for the backgrounded process to exit.
381+
// SSH connection hangs are protected by ConnectTimeout (see buildSshArgs in this file).
381382
const result = await execBuffered(this, spawnCommand, {
382383
cwd: "/", // cwd doesn't matter, we cd in the wrapper
383-
timeout: 30, // Generous timeout for SSH connection + shell startup
384384
});
385385

386386
if (result.exitCode !== 0) {

0 commit comments

Comments
 (0)