From 630f286425d7cbd83fb1ebb1a058876c9041c735 Mon Sep 17 00:00:00 2001 From: Luke Weiler Date: Thu, 12 May 2022 12:33:33 -0400 Subject: [PATCH] added try catch on yaml.safeload in shell (#1988) --- apps/shell/app.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/apps/shell/app.js b/apps/shell/app.js index ba9c541761..20ed27ce46 100644 --- 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