@tiledesk/tiledesk-server 2.3.64 → 2.3.65

package/CHANGELOG.md

@@ -1,12 +1,29 @@
 
 
-💥 TILEDESK SERVER v2.3.61 💥
+💥 TILEDESK SERVER v2.3.65 💥
 🚀        TAGGED AND PUBLISHED ON NPM           🚀
 🚀        IN PRODUCTION                        🚀
-(https://www.npmjs.com/package/@tiledesk/tiledesk-server/v/2.3.61) 
+(https://www.npmjs.com/package/@tiledesk/tiledesk-server/v/2.3.65) 
+
+# 2.3.65
+- Updated tiledesk-tybot-connector 0.1.38
+- chatbot and subscription can use send email endpoint
+- Added EMAIL_CC_ENABLED env variable to disable CCs email inboud notification
+
+# 2.3.64
+- log fix
+- Tybot updated to 0.1.37
 
 # 2.3.63
 - Tybot updated to 0.1.36
+- ChangeLog degli ultimi due commit:
+- added trainingService.js file 
+- added certified, mainCategory and intentsEngine fields for faq_kb model 
+- deleted new_bot_name for forked chatbot 
+- updated faq and faq_kb tests
+- edit trainingService.js
+- add test for bot with intentsEngine (training) 
+
 
 # 2.3.62
 - if (request.markModified) fix webhook in queue

package/app.js

@@ -489,7 +489,7 @@ app.use('/:projectid/labels', [fetchLabels],labels);
 
 app.use('/:projectid/campaigns',[passport.authenticate(['basic', 'jwt'], { session: false }), validtoken, roleChecker.hasRole('agent')], campaigns);
 
-app.use('/:projectid/emails',[passport.authenticate(['basic', 'jwt'], { session: false }), validtoken, roleChecker.hasRole('agent')], email);
+app.use('/:projectid/emails',[passport.authenticate(['basic', 'jwt'], { session: false }), validtoken, roleChecker.hasRoleOrTypes('agent', ['bot','subscription'])], email);
 
 
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tiledesk/tiledesk-server",
   "description": "The Tiledesk server module",
-  "version": "2.3.64",
+  "version": "2.3.65",
   "scripts": {
     "start": "node ./bin/www",
     "pretest": "mongodb-runner start",
@@ -42,7 +42,7 @@
     "@tiledesk/tiledesk-chatbot-util": "^0.8.33",
     "@tiledesk/tiledesk-json-rules-engine": "^4.0.3",
     "@tiledesk/tiledesk-rasa-connector": "^1.0.10",
-    "@tiledesk/tiledesk-tybot-connector": "^0.1.37",
+    "@tiledesk/tiledesk-tybot-connector": "^0.1.38",
     "@tiledesk/tiledesk-dialogflow-connector": "^1.7.4",
     "app-root-path": "^3.0.0",
     "bcrypt-nodejs": "0.0.3",

package/routes/faq_kb.js

@@ -402,7 +402,7 @@ router.post('/fork/:id_faq_kb', async (req, res) => {
 router.post('/importjson/:id_faq_kb', upload.single('uploadFile'), (req, res) => {
 
   let id_faq_kb = req.params.id_faq_kb;
-  winston.debug('id_faq_kb: ', id_faq_kb);
+  winston.info('import on id_faq_kb: ', id_faq_kb);
 
   let json_string;
   let json;
@@ -413,8 +413,6 @@ router.post('/importjson/:id_faq_kb', upload.single('uploadFile'), (req, res) =>
     json = req.body;
   }
 
-  winston.info("json_string: " + json_string);
-
   if (req.query.intentsOnly == "true") {
 
     winston.info("intents only")
@@ -473,44 +471,15 @@ router.post('/importjson/:id_faq_kb', upload.single('uploadFile'), (req, res) =>
 
   } else {
 
-    Faq_kb.findById(id_faq_kb, (err, faq_kb) => {
-      if (err) {
-        winston.error("GET FAQ-KB ERROR", err);
-        return res.status(500).send({ success: false, msg: "Error getting bot." });
-      }
-      if (!faq_kb) {
-        return res.status(404).send({ success: false, msg: 'Bot not found.' });
-      }
-
-      const json = JSON.parse(json_string);
-
-      if (json.webhook_enabled) {
-        faq_kb.webhook_enabled = json.webhook_enabled;
-      }
-      if (json.webhook_url) {
-        faq_kb.webhook_url = json.webhook_url;
-      }
-      if (json.language) {
-        faq_kb.language = json.language;
-      }
-      if (json.name) {
-        faq_kb.name = json.name;
-      }
-      if (json.description) {
-        faq_kb.description = json.description;
-      }
-
-      Faq_kb.findByIdAndUpdate(id_faq_kb, faq_kb, { new: true }, (err, updatedFaq_kb) => {
-        if (err) {
-          return res.status(500).send({ success: false, msg: "Error updating bot." });
-        }
-
-        botEvent.emit('faqbot.update', updatedFaq_kb);
+    if (req.query.create == 'true') {
+      faqService.create(json.name, undefined, req.projectid, req.user.id, "tilebot", json.description, json.webhook_url, json.webhook_enabled, json.language, undefined, undefined, undefined).then((savedFaq_kb) => {
+        winston.debug("saved (and imported) faq kb: ", savedFaq_kb);
+        botEvent.emit('faqbot.create', savedFaq_kb);
 
         json.intents.forEach((intent) => {
-
+  
           let new_faq = {
-            id_faq_kb: updatedFaq_kb._id,
+            id_faq_kb: savedFaq_kb._id,
             id_project: req.projectid,
             createdBy: req.user.id,
             intent_display_name: intent.intent_display_name,
@@ -523,7 +492,7 @@ router.post('/importjson/:id_faq_kb', upload.single('uploadFile'), (req, res) =>
             language: intent.language
           }
 
-          // overwrite duplicated intents
+          // TO DELETE: no used when req.query.create = 'true'
           if (req.query.overwrite == "true") {
             Faq.findOneAndUpdate({ id_faq_kb: id_faq_kb, intent_display_name: intent.intent_display_name }, new_faq, { new: true, upsert: true, rawResult: true }, (err, savingResult) => {
               if (err) {
@@ -559,12 +528,110 @@ router.post('/importjson/:id_faq_kb', upload.single('uploadFile'), (req, res) =>
           }
 
         })
+        return res.status(200).send(savedFaq_kb);
 
-        return res.send(updatedFaq_kb);
+      }).catch((err) => {
+        console.error("error saving faq_kb: ", err);
+        return res.status(500).send(err);
+      })
 
+    } else {
+
+      Faq_kb.findById(id_faq_kb, (err, faq_kb) => {
+        if (err) {
+          winston.error("GET FAQ-KB ERROR", err);
+          return res.status(500).send({ success: false, msg: "Error getting bot." });
+        }
+        if (!faq_kb) {
+          return res.status(404).send({ success: false, msg: 'Bot not found.' });
+        }
+  
+        // should be wrong
+        //const json = JSON.parse(json_string);
+  
+        if (json.webhook_enabled) {
+          faq_kb.webhook_enabled = json.webhook_enabled;
+        }
+        if (json.webhook_url) {
+          faq_kb.webhook_url = json.webhook_url;
+        }
+        if (json.language) {
+          faq_kb.language = json.language;
+        }
+        if (json.name) {
+          faq_kb.name = json.name;
+        }
+        if (json.description) {
+          faq_kb.description = json.description;
+        }
+  
+        Faq_kb.findByIdAndUpdate(id_faq_kb, faq_kb, { new: true }, (err, updatedFaq_kb) => {
+          if (err) {
+            return res.status(500).send({ success: false, msg: "Error updating bot." });
+          }
+  
+          botEvent.emit('faqbot.update', updatedFaq_kb);
+  
+          json.intents.forEach((intent) => {
+  
+            let new_faq = {
+              id_faq_kb: updatedFaq_kb._id,
+              id_project: req.projectid,
+              createdBy: req.user.id,
+              intent_display_name: intent.intent_display_name,
+              question: intent.question,
+              answer: intent.answer,
+              reply: intent.reply,
+              form: intent.form,
+              enabled: intent.enabled,
+              webhook_enabled: intent.webhook_enabled,
+              language: intent.language
+            }
+  
+            // overwrite duplicated intents
+            if (req.query.overwrite == "true") {
+              Faq.findOneAndUpdate({ id_faq_kb: id_faq_kb, intent_display_name: intent.intent_display_name }, new_faq, { new: true, upsert: true, rawResult: true }, (err, savingResult) => {
+                if (err) {
+                  winston.error("findOneAndUpdate (upsert) FAQ ERROR ", err);
+                } else {
+  
+                  if (savingResult.lastErrorObject.updatedExisting == true) {
+                    winston.info("updated existing intent")
+                    faqBotEvent.emit('faq.update', savingResult.value);
+                  } else {
+                    winston.info("new intent crated")
+                    faqBotEvent.emit('faq.create', savingResult.value);
+                  }
+  
+                }
+  
+              })
+  
+              // don't overwrite duplicated intents
+            } else {
+              Faq.create(new_faq, (err, savedFaq) => {
+                if (err) {
+                  winston.debug("create new FAQ ERROR ", err);
+                  if (err.code == 11000) {
+                    winston.error("Duplicate intent_display_name.");
+                    winston.info("Skip duplicated intent_display_name");
+                  } else {
+                    winston.info("new intent crated")
+                    faqBotEvent.emit('faq.create', savedFaq);
+                  }
+                }
+              })
+            }
+  
+          })
+  
+          return res.send(updatedFaq_kb);
+  
+        })
+  
       })
+    }
 
-    })
   }
 
 })

package/services/emailService.js

@@ -124,6 +124,13 @@ class EmailService {
     }
     winston.info('EmailService headers: ' + JSON.stringify(this.headers));
 
+    this.ccEnabled = false 
+
+    if (process.env.EMAIL_CC_ENABLED ==="true" || process.env.EMAIL_CC_ENABLED === true ) {
+      this.ccEnabled = true;
+    }
+    winston.info('EmailService ccEnabled: ' + ccEnabled);
+
   }
 
   readTemplate(templateName, settings) {
@@ -1124,14 +1131,17 @@ class EmailService {
         references = message.request.attributes.email_references;
       }        
 
-      if (message.request.attributes.email_cc) {
-        cc = message.request.attributes.email_cc;       
-      }
-      winston.debug("email message.request.attributes.email_ccStr: "+ message.request.attributes.email_ccStr);
-      if (message.request.attributes.email_ccStr!=undefined) {
-        ccString = message.request.attributes.email_ccStr;
-        winston.debug("email set ccString");
+      if (this.ccEnabled==true) {
+        if (message.request.attributes.email_cc) {
+          cc = message.request.attributes.email_cc;       
+        }
+        winston.debug("email message.request.attributes.email_ccStr: "+ message.request.attributes.email_ccStr);
+        if (message.request.attributes.email_ccStr!=undefined) {
+          ccString = message.request.attributes.email_ccStr;
+          winston.debug("email set ccString");
+        }
       }
+      
     }
     winston.verbose("email inReplyTo: "+ inReplyTo);
     winston.verbose("email references: "+ references);
@@ -1293,13 +1303,15 @@ class EmailService {
         references = message.request.attributes.email_references;
       }        
 
-      if (message.request.attributes.email_cc) {
-        cc = message.request.attributes.email_cc;       
-      }
-      winston.debug("email message.request.attributes.email_ccStr: "+ message.request.attributes.email_ccStr);
-      if (message.request.attributes.email_ccStr!=undefined) {
-        ccString = message.request.attributes.email_ccStr;
-        winston.debug("email set ccString");
+      if (this.ccEnabled==true) {
+        if (message.request.attributes.email_cc) {
+          cc = message.request.attributes.email_cc;       
+        }
+        winston.debug("email message.request.attributes.email_ccStr: "+ message.request.attributes.email_ccStr);
+        if (message.request.attributes.email_ccStr!=undefined) {
+          ccString = message.request.attributes.email_ccStr;
+          winston.debug("email set ccString");
+        }
       }
     }
     winston.verbose("email inReplyTo: "+ inReplyTo);

package/test/example.json

@@ -0,0 +1 @@
+{"webhook_enabled":false,"language":"en","name":"examplebot","intents":[{"webhook_enabled":false,"enabled":true,"question":"\\start","answer":"Hi","intent_display_name":"start","language":"en"},{"webhook_enabled":false,"enabled":true,"question":"question1","answer":"question1","intent_display_name":"question1","language":"en"},{"webhook_enabled":false,"enabled":true,"question":"defaultFallback","answer":"I can not provide an adequate answer. Write a new question or talk to a human agent.\n* Back to start tdAction:start\n* See the docs https://docs.tiledesk.com/\n* 👨🏻‍🦰 I want an agent","intent_display_name":"defaultFallback","language":"en"}]}

package/test/faqkbRoute.js

@@ -232,6 +232,50 @@ describe('FaqKBRoute', () => {
             })
         })
 
+        it('create bot and import json', (done) => {
+
+            var email = "test-signup-" + Date.now() + "@email.com";
+            var pwd = "pwd";
+
+            userService.signup(email, pwd, "Test Firstname", "Test lastname").then(function (savedUser) {
+                projectService.create("test-faqkb-create", savedUser._id).then(function (savedProject) {
+
+                    chai.request(server)
+                        .post('/' + savedProject._id + '/faq_kb/importjson/' + null + "?create=true")
+                        .auth(email, pwd)
+                        .set('Content-Type', 'text/plain')
+                        .attach('uploadFile', fs.readFileSync(path.resolve(__dirname, './example.json')), 'example.json')
+                        .end((err, res) => {
+                            if (log) {
+                                console.log("import json res: ", res.body);
+                            }
+                            res.should.have.status(200);
+                            res.should.be.a('object');
+                            expect(res.body.name).to.equal("examplebot");
+                            expect(res.body.language).to.equal("en");
+
+                            let id_faq_kb = res.body._id;
+
+                            chai.request(server)
+                                .get('/' + savedProject._id + '/faq?id_faq_kb=' + id_faq_kb)
+                                .auth(email, pwd)
+                                .end((err, res) => {
+                                    if (log) {
+                                        console.log("faq_list: ", res.body);
+                                    }
+                                    res.should.have.status(200);
+                                    res.body.should.be.an('array').that.is.not.empty;
+
+                                    done();
+
+                                })
+                        })
+
+                })
+            })
+
+        })
+
 
         it('import json (overwrite true)', (done) => {
 
@@ -259,7 +303,7 @@ describe('FaqKBRoute', () => {
                                 .post('/' + savedProject._id + '/faq_kb/importjson/' + id_faq_kb + "?overwrite=true")
                                 .auth(email, pwd)
                                 .set('Content-Type', 'text/plain')
-                                .attach('uploadFile', fs.readFileSync(path.resolve(__dirname, './example-json.txt')), 'example-json.txt')
+                                .attach('uploadFile', fs.readFileSync(path.resolve(__dirname, './example.json')), 'example.json')
                                 .end((err, res) => {
                                     if (log) {
                                         console.log("import json res: ", res.body);