@tiledesk/tiledesk-server 2.10.9 → 2.10.11

package/CHANGELOG.md

@@ -5,6 +5,11 @@
 🚀        IN PRODUCTION                        🚀
 (https://www.npmjs.com/package/@tiledesk/tiledesk-server/v/2.3.77) 
 
+# 2.10.11
+- Updated tybot-connector to 0.2.118
+- Removed logs
+- Removed $clusterTime from add kb response
+
 # 2.10.9
 - Removed logs
 - Updated tiledesk-apps to 1.0.26

package/app.js

@@ -458,11 +458,11 @@ var projectSetter = function (req, res, next) {
       if (err) {
         winston.warn("Problem getting project with id: " + projectid + " req.originalUrl:  " + req.originalUrl);
       }
-
       winston.debug("projectSetter project:" + project);
       if (!project) {
         winston.warn("ProjectSetter project not found with id: " + projectid);
-        next();
+        //next();
+        return res.status(400).send({ error: "Project not found with id: " + projectid })
       } else {
         req.project = project;
         next(); //call next one time for projectSetter function

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tiledesk/tiledesk-server",
   "description": "The Tiledesk server module",
-  "version": "2.10.9",
+  "version": "2.10.11",
   "scripts": {
     "start": "node ./bin/www",
     "pretest": "mongodb-runner start",
@@ -48,7 +48,7 @@
     "@tiledesk/tiledesk-rasa-connector": "^1.0.10",
     "@tiledesk/tiledesk-telegram-connector": "^0.1.14",
     "@tiledesk/tiledesk-train-jobworker": "^0.0.11",
-    "@tiledesk/tiledesk-tybot-connector": "^0.2.113",
+    "@tiledesk/tiledesk-tybot-connector": "^0.2.118",
     "@tiledesk/tiledesk-whatsapp-connector": "^0.1.73",
     "@tiledesk/tiledesk-whatsapp-jobworker": "^0.0.8",
     "@tiledesk/tiledesk-sms-connector": "^0.1.10",

package/routes/email.js

@@ -387,7 +387,7 @@ router.post('/internal/send',
   let replyto = req.body.replyto;
   winston.debug("replyto: " + replyto);
 
-  winston.info("Sending an email with text : " + text + " to " + to);
+  //winston.info("Sending an email with text : " + text + " to " + to);
 
   let quoteManager = req.app.get('quote_manager');
 

package/routes/faq_kb.js

@@ -833,10 +833,10 @@ router.post('/importjson/:id_faq_kb', roleChecker.hasRole('admin'), upload.singl
 
           if (savingResult) {
             if (savingResult.lastErrorObject.updatedExisting === true) {
-              winston.info("updated existing intent")
+              winston.verbose("updated existing intent")
               faqBotEvent.emit('faq.update', savingResult.value);
             } else {
-              winston.info("new intent created")
+              winston.verbose("new intent created")
               faqBotEvent.emit('faq.create', savingResult.value);
             }
           }
@@ -849,7 +849,7 @@ router.post('/importjson/:id_faq_kb', roleChecker.hasRole('admin'), upload.singl
           let faq = await Faq.create(new_faq).catch((err) => {
             if (err.code == 11000) {
               winston.error("Duplicate intent_display_name.");
-              winston.info("Skip duplicated intent_display_name");
+              winston.verbose("Skip duplicated intent_display_name");
             } else {
               winston.error("Error creating new intent: ", err);
             }
@@ -857,7 +857,7 @@ router.post('/importjson/:id_faq_kb', roleChecker.hasRole('admin'), upload.singl
           })
 
           if (faq) {
-            winston.info("new intent created")
+            winston.verbose("new intent created")
             faqBotEvent.emit('faq.create', faq);
           }
         }

package/routes/kb.js

@@ -521,8 +521,8 @@ router.get('/namespace/:id/chunks/:content_id', async (req, res) => {
     return res.status(200).send(chunks);
 
   }).catch((err) => {
-    console.log("error getting content chunks err.response: ", err.response)
-    console.log("error getting content chunks err.data: ", err.data)
+    console.error("error getting content chunks err.response: ", err.response)
+    console.error("error getting content chunks err.data: ", err.data)
     return res.status(500).send({ success: false, error: err });
   })
 
@@ -961,6 +961,9 @@ router.post('/', async (req, res) => {
     }
     else {
 
+      delete raw.ok;
+      delete raw.$clusterTime;
+      delete raw.operationTime;
       res.status(200).send(raw);
 
       let saved_kb = raw.value;
@@ -1105,8 +1108,6 @@ router.post('/multi', upload.single('uploadFile'), async (req, res) => {
     }
   })
 
-    console.log("kbs: ", kbs);
-
   saveBulk(operations, kbs, project_id).then((result) => {
 
     let ns = namespaces.find(n => n.id === namespace_id);
@@ -1116,7 +1117,6 @@ router.post('/multi', upload.single('uploadFile'), async (req, res) => {
     resources = resources.map(({ _id, scrape_options, ...rest }) => {
       return { id: _id, webhook: webhook, parameters_scrape_type_4: scrape_options, engine: engine, ...rest}
     });
-    console.log("resources to be sent to worker: ", resources);
     winston.verbose("resources to be sent to worker: ", resources);
 
     scheduleScrape(resources);
@@ -1183,7 +1183,6 @@ router.post('/csv', upload.single('uploadFile'), async (req, res) => {
       let question = data[0];
       let answer = data[1];
 
-      console.log("data. ", data)
       kbs.push({
         id_project: project_id,
         name: question,

package/routes/request.js

@@ -744,7 +744,7 @@ router.post('/:requestid/email/send',
     winston.debug("request_id: " + request_id);
 
     let subject = req.body.subject;
-    winston.info("subject: " + subject);
+    winston.debug("subject: " + subject);
 
     winston.debug("req.project", req.project);
 

package/test/app-test.js

@@ -0,0 +1,50 @@
+process.env.NODE_ENV = 'test';
+
+let chai = require('chai');
+let chaiHttp = require('chai-http');
+let server = require('../app');
+let should = chai.should();
+const userService = require('../services/userService');
+const projectService = require('../services/projectService');
+
+let log = false;
+
+let expect = chai.expect;
+let assert = chai.assert;
+
+chai.use(chaiHttp);
+
+describe('Root Test', () => {
+
+    /**
+     * Try to perform a request with an invalid id project.
+     * A fake id or non existent mongo id (like user id) is passed.
+     * This test must respond with status 400 and with an error.
+     */
+    it("Wrong request url", (done) => {
+
+        var email = "test-signup-" + Date.now() + "@email.com";
+        var pwd = "pwd";
+
+        userService.signup(email, pwd, "Test Firstname", "Test Lastname").then((savedUser) => {
+            projectService.create("test-root-project", savedUser._id).then((savedProject) => {
+
+                let fake_id_project = "fake1234"
+
+                chai.request(server)
+                    //.get('/' + savedUser._id + '/faq_kb')
+                    .get('/' + fake_id_project + '/faq_kb')
+                    .auth(email, pwd)
+                    .end((err, res) => {
+                        if (err) { console.error("err: ", err); }
+                        if (log) { console.log("res.body", res.body); }
+                        
+                        res.should.have.status(400);
+                        assert.notEqual(res.body.error, null);
+                        expect(res.body.error).to.equal("Project not found with id: " + fake_id_project)
+                        done();
+                    })
+            })
+        })
+    }).timeout(5000)
+})