@andersbakken/fisk 5.0.2 → 5.0.3

package/daemon/fisk-daemon.js

@@ -3433,6 +3433,28 @@ var output = {
 
 var fs = libExports;
 
+// systemd socket activation: the first passed fd is at SD_LISTEN_FDS_START (3).
+// See sd_listen_fds(3). When LISTEN_PID matches our pid and LISTEN_FDS >= 1,
+// listen on the inherited fd instead of creating a new socket. This lets
+// systemd own the socket file, so its inode survives daemon restarts and any
+// bind-mount of the socket (e.g. docker -v /tmp/fisk.socket:/tmp/fisk.socket)
+// keeps working across upgrades.
+const SD_LISTEN_FDS_START = 3;
+function systemdListenFd() {
+    const pid = process.env.LISTEN_PID;
+    const fds = process.env.LISTEN_FDS;
+    if (!pid || !fds) {
+        return undefined;
+    }
+    if (parseInt(pid, 10) !== process.pid) {
+        return undefined;
+    }
+    const count = parseInt(fds, 10);
+    if (!count || count < 1) {
+        return undefined;
+    }
+    return SD_LISTEN_FDS_START;
+}
 class Server extends EventEmitter__default["default"] {
     constructor(option, common) {
         super();
@@ -3443,6 +3465,7 @@ class Server extends EventEmitter__default["default"] {
         this.option = option;
         this._connections = {};
         this._connectionId = 0;
+        this._activated = false;
     }
     close() {
         if (this.debug) {
@@ -3451,14 +3474,44 @@ class Server extends EventEmitter__default["default"] {
         if (this.server) {
             this.server.close();
         }
-        try {
-            fs.unlinkSync(this.file);
-        }
-        catch (err) {
-            /* */
+        // When socket-activated, systemd owns the socket file. Do not unlink
+        // it -- that would break the bind-mount contract for existing clients.
+        if (!this._activated) {
+            try {
+                fs.unlinkSync(this.file);
+            }
+            catch (err) {
+                /* */
+            }
         }
     }
     listen() {
+        const inheritedFd = systemdListenFd();
+        if (inheritedFd !== undefined) {
+            this._activated = true;
+            if (this.debug) {
+                console.log("Server::listen using systemd-activated fd", inheritedFd);
+            }
+            return new Promise((resolve) => {
+                let connected = false;
+                this.server = net__default["default"].createServer(this._onConnection.bind(this));
+                this.server.listen({ fd: inheritedFd }, () => {
+                    connected = true;
+                    resolve();
+                });
+                this.server.on("error", (err) => {
+                    if (!connected) {
+                        console.error("Got server error", err);
+                        setTimeout(this.listen.bind(this), 1000);
+                    }
+                });
+                this.server.on("close", () => {
+                    if (!connected) {
+                        setTimeout(this.listen.bind(this), 1000);
+                    }
+                });
+            });
+        }
         try {
             fs.unlinkSync(this.file); // this should be more
             // complicated with attempts to

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@andersbakken/fisk",
-    "version": "5.0.2",
+    "version": "5.0.3",
     "description": "Fisk, a distributed compile system",
     "scripts": {
         "lint": "eslint . --ext .ts",