@tiledesk/tiledesk-server 2.3.29 → 2.3.31

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 
-# Untagged
+
+# 2.3.31
+- logfix
+
+# 2.3.30
+- save multiple messages fix with sequential promises
+- JSON_BODY_LIMIT increased to 500KB. Added JSON_BODY_LIMIT env variable
+
+# 2.3.29
 - UIDGenerator class replacement for request route
 - Added hasRole cache for project_user
 - Added index { id_project: 1, role: 1, status: 1, createdAt: 1  } for  Project_user schema
@@ -9,9 +17,14 @@
 - Log fix
 - add fields reply and enabled to faq model 
 - field answer in faq model is no longer required
-- add field public to faq_kb model (IGNORA PER IL MOMENTO)
+- add field public to faq_kb model
 - when template is undefined the empty template is now created
-- add test
+- Add unit test for project and project_user
+- Created new endpoint for creating new requests with unit test
+- Created new endpoint for inserting multiple messages at once with unit test
+- New validation for empty text for new message endpoint 
+- Added project_user_test route 
+- Unit test fix
 
 # 2.3.28
 - UIDGenerator renamed fix
@@ -29,7 +42,7 @@
 
 # 2.3.24
 - Increased cache TTL from: standardTTL from 120 to 300, longTTL from 1200 to 3600
-- Added cache request.create.simple, + cacheEnabler. name fix
+- Added cache request.create.simple + cacheEnabler. name fix
 - Disabled unapplicable cache for updateWaitingTimeByRequestId and find request by id REST API
 - Updated dependency @tiledesk/tiledesk-tybot-connector to 0.1.14
 - Restored populateMessageWithRequest. Bug with \agent command. Try to resolve performance with cache

package/app.js

@@ -148,7 +148,6 @@ geoService.listen();
 
 
 
-
 var faqBotHandler = require('./services/faqBotHandler');
 faqBotHandler.listen();
 
@@ -225,7 +224,10 @@ if (process.env.ENABLE_ALTERNATIVE_CORS_MIDDLEWARE === "true") {
 
 // https://stackoverflow.com/questions/18710225/node-js-get-raw-request-body-using-express
 
-app.use(bodyParser.json({
+const JSON_BODY_LIMIT = process.envJSON_BODY_LIMIT || '500KB';
+winston.debug("JSON_BODY_LIMIT : " + JSON_BODY_LIMIT);
+
+app.use(bodyParser.json({limit: JSON_BODY_LIMIT,
   verify: function (req, res, buf) {
     // var url = req.originalUrl;
     // if (url.indexOf('/stripe/')) {
@@ -235,7 +237,7 @@ app.use(bodyParser.json({
   }
 }));
 
-app.use(bodyParser.urlencoded({ extended: false }));
+app.use(bodyParser.urlencoded({limit: JSON_BODY_LIMIT, extended: false }));
 
 app.use(cookieParser());
 app.use(express.static(path.join(__dirname, 'public')));

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tiledesk/tiledesk-server",
   "description": "The Tiledesk server module",
-  "version": "2.3.29",
+  "version": "2.3.31",
   "scripts": {
     "start": "node ./bin/www",
     "pretest": "mongodb-runner start",

package/routes/message.js

@@ -21,6 +21,7 @@ const { check, validationResult } = require('express-validator');
 
 var Project_user = require("../models/project_user");
 var mongoose = require('mongoose');
+var PromiseUtil = require("../utils/promiseUtil");
 
 csv = require('csv-express');
 csv.separator = ';';
@@ -406,7 +407,8 @@ async (req, res)  => {
       });
       winston.debug('--- >promises', promises);
 
-      Promise.all(promises).then((values) => {
+      //Promise.all(promises).then((values) => {
+      PromiseUtil.doAllSequentually(promises).then((values) => {
         winston.info('Inserted multiple messages with values: ', values);
         return res.status(200).json(values);
       }).catch((err) => {

package/services/requestService.js

@@ -513,7 +513,7 @@ class RequestService {
            try {
             //  getOperators(departmentid, projectid, nobot, disableWebHookCall, context) {
               var result = await departmentService.getOperators(departmentid, id_project, false, undefined, context);
-              console.log("************* after get operator: "+new Date().toISOString());
+              // console.log("************* after get operator: "+new Date().toISOString());
 
               winston.debug("getOperators", result);
            } catch(err) {

package/utils/promiseUtil.js

@@ -0,0 +1,31 @@
+class PromiseUtil {
+
+  async doAllSequentually(promises) {
+    // return new Promise(async function (resolve, reject) {         
+    // console.log("promises",promises);
+    var values = [];
+    for (let i=0; i < promises.length; i++) {
+        // console.log("promises[i]",promises[i]);
+
+    //   const val = await 
+      var value = await promises[i]
+    //   .then(function(value) {
+    //     console.log("done promises[i]",value);
+    //     values.push(value);
+    //     console.log("values before",values);
+    //   });
+        values.push(value);
+    };
+
+    console.log("values",values);
+    // return resolve(values);
+    
+    // });
+    return values;
+  }
+}
+
+var promiseUtil = new PromiseUtil();
+
+
+module.exports = promiseUtil;