external-ips 0.0.4 → 0.0.5

package/index.js

@@ -10,7 +10,6 @@ function normalizeFamily(family){
 		return false;
 	} else throw new Error('bad family='+family);
 }
-
 class AbstractObject {
 	constructor(family){
 		assert(this.constructor!==AbstractObject); //Abstract
@@ -29,9 +28,15 @@ class AbstractObject {
 			if((!family || !self.family || self.family===family) && !options.localAddress){
 				const ip = self.random();
 				if(ip){
-					//console.log('++++++', ip, ip.address);
-					options.localAddress = ip.address ? ip.address : ip;
-				} //else console.log('++++++---', ip);
+					//КОПИЯ, а не мутация. Agent уже вычислил ключ реестра (getName БЕЗ localAddress)
+					//и замкнул этот же объект options в свои слушатели. Если дописать localAddress
+					//в него, removeSocket/onFree пересчитают имя уже С адресом и попадут в другой
+					//список: закрытые сокеты никогда не удаляются из agent.sockets (на боевом
+					//mexc-гейтвее агент накопил 7744 сокета, из них 5578 мёртвых, ~1.5 ГБ/ч), а
+					//freeSockets кладутся под ключ, по которому их никто не ищет — keep-alive не
+					//переиспользуется вовсе, и каждый запрос открывает новый TLS.
+					return old.call(agent, { ...options, localAddress: ip.address ? ip.address : ip }, callback);
+				}
 			}
 			return old.call(agent, options, callback);
 		}

package/package.json

@@ -1,27 +1,29 @@
 {
-    "name": "external-ips",
-    "version": "0.0.4",
-    "main": "./index.js",
-    "scripts": {
-        "test": "echo \"Error: no test specified\" && exit 1"
-    },
-    "author": "Andrey Belousoff <a.v.belousoff@gmail.com>",
-    "license": "MIT",
-    "description": "any IPs in HTTP(s) requests or WS",
-    "keywords": ["ips", "ip"],
-    "dependencies": {
-    },
-    "engines": {
-        "node": ">=0.1.0"
-    },
-    "repository": {
-        "type": "git",
-        "url": "https://github.com/Hkey1/external-ips"
-    },
-    "readme":"External IPs",
-    "readmeFilename": "README.md",
-    "bugs": {
-        "url": "https://github.com/Hkey1/external-ips/issues"
-    },
-    "homepage": "https://github.com/Hkey1/external-ips"
+  "name": "external-ips",
+  "version": "0.0.5",
+  "main": "./index.js",
+  "scripts": {
+    "test": "node test.js"
+  },
+  "author": "Andrey Belousoff <a.v.belousoff@gmail.com>",
+  "license": "MIT",
+  "description": "any IPs in HTTP(s) requests or WS",
+  "keywords": [
+    "ips",
+    "ip"
+  ],
+  "dependencies": {},
+  "engines": {
+    "node": ">=0.1.0"
+  },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/Hkey1/external-ips"
+  },
+  "readme": "External IPs",
+  "readmeFilename": "README.md",
+  "bugs": {
+    "url": "https://github.com/Hkey1/external-ips/issues"
+  },
+  "homepage": "https://github.com/Hkey1/external-ips"
 }

package/test.js

@@ -0,0 +1,34 @@
+//node test.js — регрессия: патч не должен ломать учёт сокетов у http.Agent
+const assert = require("node:assert");
+const http = require("node:http");
+const IPs = require("./index.js");
+
+const server = http.createServer((req, res)=>res.end("ok"));
+server.listen(0, "127.0.0.1", async()=>{
+    const port = server.address().port;
+    try {
+        const agent = new http.Agent({ keepAlive: true });
+        //как manyIPs в ccxtPatch; random() вернёт локальный адрес этой машины
+        IPs.v4.patchAgent(agent);
+        for(let i = 0; i < 8; i++){
+            await new Promise((res, rej)=>{
+                http.get({ host: "127.0.0.1", port, agent }, r=>{ r.resume(); r.on("end", res); }).on("error", rej);
+            });
+            await new Promise(r=>setTimeout(r, 30));
+        }
+        const cnt = o=>Object.values(o).reduce((a, arr)=>a + arr.length, 0);
+        const busy = cnt(agent.sockets);
+        const free = cnt(agent.freeSockets);
+        const dead = Object.values(agent.sockets).flat().filter(s=>s.destroyed).length;
+        console.log(`sockets=${busy} (мёртвых ${dead}), freeSockets=${free}`);
+        //до фикса: sockets=8 под одним ключом, freeSockets=8 под другим, реюза ноль
+        assert.equal(busy, 0, `в agent.sockets не должно оставаться сокетов, осталось ${busy}`);
+        assert.equal(dead, 0);
+        assert.ok(free >= 1 && free <= 2, `keep-alive должен переиспользоваться: freeSockets=${free}`);
+        console.log("ok: учёт агента цел, keep-alive переиспользуется");
+        agent.destroy(); server.close();
+    } catch(err){
+        console.error("FAIL:", err.message);
+        server.close(); process.exitCode = 1;
+    }
+});