hotfix vrps file reloading memory leak

nttgin · Jul 20, 2020 · 9bf2227 · 9bf2227
1 parent 78f0088
commit 9bf2227
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 7 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -52,7 +52,7 @@
     "nodemailer": "^6.4.10",
     "path": "^0.12.7",
     "restify": "^8.5.1",
-    "rpki-validator": "^2.1.5",
+    "rpki-validator": "^2.1.6",
     "semver": "^7.3.2",
     "syslog-client": "^1.1.1",
     "ws": "^7.3.1",

diff --git a/src/monitors/monitorRPKI.js b/src/monitors/monitorRPKI.js
@@ -110,16 +110,21 @@ export default class MonitorRPKI extends Monitor {
                             clearInterval(this.validationTimer); // Stop validation cycle
                         }
 
+                        if (this.rpki) {
+                            this.rpki.destroy();
+                        }
+
                         this.rpki = new rpki({
                             connector: "external",
-                            clientId: env.clientId,
-                            vrps
+                            clientId: env.clientId
                         });
 
+                        this.rpki.setVRPs(vrps);
+
                         this.rpki
                             .preCache()
                             .then(() => {
-                                this.validationTimer = setInterval(this.validateBatch, 100); // If already cached, we can validate more often
+                                this.validationTimer = setInterval(this.validateBatch, 50); // If already cached, we can validate more often
                             })
                             .catch(() => {
                                 this.logger.log({