From d6570d1496a42a9730e9b1112200c09094d5ddd9 Mon Sep 17 00:00:00 2001 From: psychobunny Date: Mon, 14 Oct 2013 12:00:31 -0400 Subject: [PATCH] prevent admin crashes on improperly configured plugins --- src/plugins.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/plugins.js b/src/plugins.js index 2fe845a56c..28ddaae55e 100644 --- a/src/plugins.js +++ b/src/plugins.js @@ -268,7 +268,13 @@ var fs = require('fs'), fs.readFile(path.join(file, 'plugin.json'), next); }, function(configJSON, next) { - var config = JSON.parse(configJSON); + try { + var config = JSON.parse(configJSON); + } catch (err) { + winston.warn("Plugin: " + file + " is corrupted or invalid. Please check plugin.json for errors.") + return next(err, null); + } + _self.isActive(config.id, function(err, active) { if (err) next(new Error('no-active-state'));