Skip to content

Commit

Permalink
Fixes Haufe-Lexware/wicked.haufe.io#73 - add guard for non-existing c…
Browse files Browse the repository at this point in the history
…hatbot hooks
  • Loading branch information
DonMartin76 committed Jul 18, 2017
1 parent 58527bd commit 3e7e504
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions routes/chatbot.js
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,10 @@ router.get('/', function (req, res, next) {
var envVars = utils.loadEnvDict(req.app);
utils.mixinEnv(glob, envVars);

if (!glob.chatbot.hookUrls) {
glob.chatbot.hookUrls = [];
}

res.render('chatbot',
{
configPath: req.app.get('config_path'),
Expand All @@ -26,14 +30,17 @@ router.post('/', function (req, res, next) {
var envVars = utils.loadEnvDict(req.app);
glob.chatbot = body.glob.chatbot;
utils.mixoutEnv(glob, envVars);

if ("deleteHook" == body.__action) {
var index = Number(body.__object);
glob.chatbot.hookUrls.splice(index, 1);
} else if ("addHook" == body.__action) {
if (!glob.chatbot.hookUrls) {
glob.chatbot.hookUrls = [];
}
glob.chatbot.hookUrls.push('https://url.to.your.slack/hookidentifierwhichisasecrect');
}

utils.saveGlobals(req.app, glob);
utils.saveEnvDict(req.app, envVars, "default");

Expand Down

0 comments on commit 3e7e504

Please sign in to comment.