dynamic-self-register-proxy 1.0.5 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dynamic-self-register-proxy",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "Dynamic reverse proxy with self-registration API - applications can register themselves and receive an automatically assigned port",
   "main": "proxy-client.js",
   "bin": {

package/proxy.js

@@ -818,22 +818,51 @@ function escapeHtml(text) {
 }
 
 // ============================================
-// PROXY MIDDLEWARE
+// PROXY MIDDLEWARE (PERSISTENT)
 // ============================================
 
 /**
- * Router dynamique - détermine la cible en fonction de la requête
+ * Middleware de proxy persistant pour éviter les fuites de mémoire (EventEmitter MaxListenersExceededWarning)
+ * Créé une seule fois et réutilisé pour toutes les requêtes.
  */
-const dynamicRouter = (req) => {
-  const route = findRouteForPath(req.path);
-  if (route) {
-    return `http://localhost:${route.port}`;
+const persistentProxyMiddleware = createProxyMiddleware({
+  target: 'http://localhost', // Cible par défaut (sera surchargée par router)
+  router: (req) => req.proxyTarget,
+  changeOrigin: true,
+  pathRewrite: (path, req) => {
+    const routePath = req.proxyRoutePath;
+    if (routePath && path.startsWith(routePath)) {
+      return path.slice(routePath.length) || '/';
+    }
+    return path;
+  },
+  on: {
+    proxyReq: (proxyReq, req) => {
+      // Ré-écriture du body si présent (nécessaire car express.json() a consommé le stream)
+      if (req.body && typeof req.body === 'object' && Object.keys(req.body).length > 0) {
+        const bodyData = JSON.stringify(req.body);
+        proxyReq.setHeader('Content-Type', 'application/json');
+        proxyReq.setHeader('Content-Length', Buffer.byteLength(bodyData));
+        proxyReq.write(bodyData);
+      }
+      console.log(`[PROXY] ${req.method} ${req.path} -> ${req.proxyTarget}`);
+    },
+    error: (err, req, res) => {
+      // Si la réponse a déjà été envoyée, on ne fait rien
+      if (res.headersSent) return;
+      
+      console.error(`[PROXY ERROR] ${req.path}:`, err.message);
+      res.status(502).json({
+        error: 'Proxy error',
+        message: err.message,
+        target: req.proxyTarget
+      });
+    }
   }
-  return null;
-};
+});
 
 /**
- * Middleware de proxy pour toutes les autres requêtes
+ * Middleware principal qui gère le routage vers le proxy
  */
 app.use((req, res, next) => {
   // Ignore les routes de l'API proxy
@@ -841,9 +870,9 @@ app.use((req, res, next) => {
     return next();
   }
 
-  const target = dynamicRouter(req);
+  const route = findRouteForPath(req.path);
   
-  if (!target) {
+  if (!route) {
     return res.status(404).json({
       error: 'No route registered for this path',
       path: req.path,
@@ -851,45 +880,15 @@ app.use((req, res, next) => {
     });
   }
 
+  // Stocke les informations de la route dans l'objet req pour le middleware persistant
+  req.proxyTarget = `http://localhost:${route.port}`;
+  
   // Trouve le path de la route pour le pathRewrite
-  const route = findRouteForPath(req.path);
-  const routePath = [...registry.routes.entries()]
+  req.proxyRoutePath = [...registry.routes.entries()]
     .find(([, v]) => v === route)?.[0];
 
-  const proxyMiddleware = createProxyMiddleware({
-    target,
-    changeOrigin: true,
-    pathRewrite: (path) => {
-      // Retire le préfixe de la route du path
-      if (routePath && path.startsWith(routePath)) {
-        const newPath = path.slice(routePath.length) || '/';
-        return newPath;
-      }
-      return path;
-    },
-    on: {
-      proxyReq: (proxyReq, req) => {
-        // Ré-écriture du body si présent (nécessaire car express.json() a consommé le stream)
-        if (req.body && Object.keys(req.body).length > 0) {
-          const bodyData = JSON.stringify(req.body);
-          proxyReq.setHeader('Content-Type', 'application/json');
-          proxyReq.setHeader('Content-Length', Buffer.byteLength(bodyData));
-          proxyReq.write(bodyData);
-        }
-        console.log(`[PROXY] ${req.method} ${req.path} -> ${target}`);
-      },
-      error: (err, req, res) => {
-        console.error(`[PROXY ERROR] ${req.path}:`, err.message);
-        res.status(502).json({
-          error: 'Proxy error',
-          message: err.message,
-          target
-        });
-      }
-    }
-  });
-
-  proxyMiddleware(req, res, next);
+  // Délègue au middleware de proxy persistant
+  persistentProxyMiddleware(req, res, next);
 });
 
 // ============================================