Skip to content

Commit 5095745

Browse files
committed
🤖 Use SIGKILL instead of SIGTERM for SSH timeout
SSH processes weren't being killed immediately with SIGTERM, causing timeout tests to fail. Use SIGKILL (like LocalRuntime does) to ensure immediate termination when timeout is reached.
1 parent ac11bc0 commit 5095745

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/runtime/SSHRuntime.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -183,14 +183,14 @@ export class SSHRuntime implements Runtime {
183183

184184
// Handle abort signal
185185
if (options.abortSignal) {
186-
options.abortSignal.addEventListener("abort", () => sshProcess.kill());
186+
options.abortSignal.addEventListener("abort", () => sshProcess.kill("SIGKILL"));
187187
}
188188

189189
// Handle timeout
190190
if (options.timeout !== undefined) {
191191
setTimeout(() => {
192192
timedOut = true;
193-
sshProcess.kill();
193+
sshProcess.kill("SIGKILL");
194194
}, options.timeout * 1000);
195195
}
196196

0 commit comments

Comments
 (0)