Merge pull request #1434 from evgenykireev/fixing-plugins-env-js

do not use hjson for plugins.env.js
This commit is contained in:
Wyatt Johnson
2018-03-13 17:03:18 -06:00
committed by GitHub
2 changed files with 14 additions and 10 deletions
+2 -1
View File
@@ -1 +1,2 @@
module.exports = JSON.parse(process.env.TALK_PLUGINS_JSON);
const hjson = require('hjson');
module.exports = hjson.parse(process.env.TALK_PLUGINS_JSON);
+12 -9
View File
@@ -26,17 +26,20 @@ try {
if (PLUGINS_JSON && PLUGINS_JSON.length > 0) {
debug('Now using TALK_PLUGINS_JSON environment variable for plugins');
pluginsPath = envPlugins;
} else if (fs.existsSync(customPlugins)) {
debug(`Now using ${customPlugins} for plugins`);
pluginsPath = customPlugins;
plugins = require(pluginsPath);
} else {
debug(`Now using ${defaultPlugins} for plugins`);
pluginsPath = defaultPlugins;
}
if (fs.existsSync(customPlugins)) {
debug(`Now using ${customPlugins} for plugins`);
pluginsPath = customPlugins;
} else {
debug(`Now using ${defaultPlugins} for plugins`);
pluginsPath = defaultPlugins;
}
// Load/parse the plugin content using hjson.
const pluginContent = fs.readFileSync(pluginsPath, 'utf8');
plugins = hjson.parse(pluginContent);
// Load/parse the plugin content using hjson.
const pluginContent = fs.readFileSync(pluginsPath, 'utf8');
plugins = hjson.parse(pluginContent);
}
} catch (err) {
if (err.code === 'ENOENT') {
console.error(