Skip to content

Commit c16b17b

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 3d1faef commit c16b17b

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
@@ -184,14 +184,14 @@ export class SSHRuntime implements Runtime {
184184

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

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

0 commit comments

Comments
 (0)