From fe6735824c97938b1d07b9ba01f37ae8232d20bc Mon Sep 17 00:00:00 2001 From: Louis Lam Date: Fri, 17 Nov 2023 16:03:52 +0800 Subject: [PATCH] Close terminal if there is no clients connecting --- backend/dockge-server.ts | 6 ++++++ backend/terminal.ts | 25 ++++++++++++++++++++++++- 2 files changed, 30 insertions(+), 1 deletion(-) diff --git a/backend/dockge-server.ts b/backend/dockge-server.ts index ca081b5..75fbdd5 100644 --- a/backend/dockge-server.ts +++ b/backend/dockge-server.ts @@ -30,6 +30,7 @@ import { Cron } from "croner"; import gracefulShutdown from "http-graceful-shutdown"; import User from "./models/user"; import childProcess from "child_process"; +import { Terminal } from "./terminal"; export class DockgeServer { app : Express; @@ -230,6 +231,11 @@ export class DockgeServer { }); + if (isDev) { + setInterval(() => { + log.debug("terminal", "Terminal count: " + Terminal.getTerminalCount()); + }, 10000); + } } async afterLogin(socket : DockgeSocket, user : User) { diff --git a/backend/terminal.ts b/backend/terminal.ts index 2a31f17..d851b85 100644 --- a/backend/terminal.ts +++ b/backend/terminal.ts @@ -34,6 +34,8 @@ export class Terminal { protected _rows : number = TERMINAL_ROWS; protected _cols : number = TERMINAL_COLS; + protected keepAliveInterval? : NodeJS.Timeout; + constructor(server : DockgeServer, name : string, file : string, args : string | string[], cwd : string) { this.server = server; this._name = name; @@ -101,10 +103,26 @@ export class Terminal { Terminal.terminalMap.delete(this.name); log.debug("Terminal", "Terminal " + this.name + " exited with code " + res.exitCode); + clearInterval(this.keepAliveInterval); + if (this.callback) { this.callback(res.exitCode); } }); + + // Close if there is no clients + this.keepAliveInterval = setInterval(() => { + const clients = this.server.io.sockets.adapter.rooms.get(this.name); + const numClients = clients ? clients.size : 0; + + if (numClients === 0) { + log.debug("Terminal", "Terminal " + this.name + " has no client, closing..."); + this.close(); + } else { + log.debug("Terminal", "Terminal " + this.name + " has " + numClients + " client(s)"); + } + }, 60 * 1000); + } public onExit(callback : (exitCode : number) => void) { @@ -138,7 +156,8 @@ export class Terminal { } close() { - this._ptyProcess?.kill(); + // Send Ctrl+C to the terminal + this.ptyProcess?.write("\x03"); } /** @@ -173,6 +192,10 @@ export class Terminal { terminal.start(); }); } + + public static getTerminalCount() { + return Terminal.terminalMap.size; + } } /**