Merge pull request #12 from eco-stake/memory-bugfix

Memory leak bugfix
eco-stake · Apr 28, 2022 · 6259752 · 6259752
2 parents 9d838bb + 5ca136f
commit 6259752
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/ecosystem.production.json b/ecosystem.production.json
@@ -2,7 +2,8 @@
   "name": "app",
   "script": "app.js",
   "instances": 5,
-  "cron_restart": "0 */3 * * *"
+  "cron_restart": "0 */3 * * *",
+  "max_memory_restart": "200M"
 }, {
   "name": "worker",
   "script": "worker.js",

diff --git a/proxy/proxyController.js b/proxy/proxyController.js
@@ -186,9 +186,10 @@ const ProxyController = (client, registry) => {
         resolve()
       })
 
-      proxy.on('proxyRes', (proxyRes, req, res) => {
-        proxyRes.headers["x-forwarded-to"] = ctx.state.proxyUrl;
-      });
+      // Causes major memory leak
+      // proxy.on('proxyRes', (proxyRes, req, res) => {
+      //   proxyRes.headers["x-forwarded-to"] = ctx.state.proxyUrl;
+      // });
 
       proxy.web(ctx.req, ctx.res, opts, e => {
         const status = {