fix: Replaced yamljs with yaml due to memory leak Unitech#5721

SeverinAlexB · Jan 5, 2024 · 83a3269 · 83a3269
1 parent c4032e2
commit 83a3269
Show file tree

Hide file tree

Showing 3 changed files with 28 additions and 35 deletions.
diff --git a/lib/Common.js b/lib/Common.js
@@ -304,7 +304,7 @@ Common.getConfigFileCandidates = function (name) {
  * @return {Object} config object
  */
 Common.parseConfig = function(confObj, filename) {
-  var yamljs = require('yamljs');
+  var YAML = require('yaml');
   var vm     = require('vm');
 
   var isConfigFile = Common.isConfigFile(filename);
@@ -323,7 +323,7 @@ Common.parseConfig = function(confObj, filename) {
     });
   }
   else if (isConfigFile == 'yaml') {
-    return yamljs.parse(confObj.toString());
+    return YAML.parse(confObj.toString());
   }
   else if (isConfigFile == 'js' || isConfigFile == 'mjs') {
     var confPath = require.resolve(path.resolve(filename));

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -196,7 +196,7 @@
     "source-map-support": "0.5.21",
     "sprintf-js": "1.1.2",
     "vizion": "~2.2.1",
-    "yamljs": "0.3.0"
+    "yaml": "^2.3.4"
   },
   "optionalDependencies": {
     "pm2-sysmonit": "^1.2.8"