fix(client): the UNIX domain socket path compatibilities to nodejs before v22

Author: fenying

CHANGES.md

@@ -1,5 +1,12 @@
 # Changes
 
+## v0.2.7
+
+- fix(client): the UNIX domain socket path should work as expected.
+
+    The UDS path was mistakenly ignored when `forceNewConnection` is set to
+    `true` on HTTP connections.
+
 ## v0.2.6
 
 - fix(client): the UNIX domain socket path should work as expected.

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@litert/websocket",
-  "version": "0.2.6",
+  "version": "0.2.7",
   "description": "The websocket protocol library for LiteRT.",
   "main": "lib/index.js",
   "scripts": {

package.json

@@ -346,13 +346,18 @@ export function createClient(opts: IWsConnectOptions): D.IClient {
 
         opts.agent = undefined;
 
-        opts.createConnection = (opts, onCreated) => {
-
-            const socketOpts: NodeNet.NetConnectOpts = {
+        opts.createConnection = (connArgs, onCreated) => {
 
-                ...opts,
-                port: Number(opts.port ?? 80),
-                host: opts.hostname ?? opts.host ?? 'localhost',
+            const socketOpts: NodeNet.NetConnectOpts = opts.socketPath ? {
+                'port': undefined,
+                'host': undefined,
+                'timeout': opts.connectTimeout ?? D.DEFAULT_CONNECT_TIMEOUT,
+                'path': opts.socketPath,
+            } : {
+                'timeout': opts.connectTimeout ?? D.DEFAULT_CONNECT_TIMEOUT,
+                'port': Number(connArgs.port ?? 80),
+                'host': connArgs.hostname ?? connArgs.host ?? 'localhost',
+                'path': undefined,
             };
 
             const socket = NodeNet.connect(socketOpts);