added try catch on yaml.safeload in shell (#1988)

OSC · May 12, 2022 · 630f286 · 630f286
1 parent f743c73
commit 630f286
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/apps/shell/app.js b/apps/shell/app.js
@@ -80,7 +80,11 @@ if (process.env.OOD_SSHHOST_ALLOWLIST){
 
 let default_sshhost, first_available_host;
 glob.sync(path.join((process.env.OOD_CLUSTERS || '/etc/ood/config/clusters.d'), '*.y*ml'))
-  .map(yml => yaml.safeLoad(fs.readFileSync(yml)))
+  .map(yml => {
+    try {
+      return yaml.safeLoad(fs.readFileSync(yml));
+    } catch(err) { /** just keep going. dashboard should have an alert about it */}
+  })
   .filter(config => (config.v2 && config.v2.login && config.v2.login.host) && ! (config.v2 && config.v2.metadata && config.v2.metadata.hidden))
   .forEach((config) => {
     let host = config.v2.login.host; //Already did checking above