@tiledesk/tiledesk-server 2.2.2 → 2.2.8

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+# 2.2.8
+- Public trigger module
+
+# 2.2.6
+- Quota license fix
+
+# 2.2.4
+- email invitation fix
+
+# 2.2.3
+- Email inboud fix (others disabled and inboudDomain variable fix and token query string encode fix)
+
+# 2.2.2
+- log fix
+
+# 2.2.1
+- log fix
 
 # 2.2.0
 - Cache circleci fix
@@ -78,8 +95,10 @@
 - Added email notification for new message and new request for email and form channel (ticket) 
 - Added microLanguageTransformationInterceptor enabled when message.attributes.microlanguage==true
 
+# 2.1.40.35
+- Quota license fix
 
-# 2.1.40.34 -> PROD
+# 2.1.40.34
 - logfix
 
 # 2.1.40.33

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@tiledesk/tiledesk-server",
   "description": "The Tiledesk server module",
-  "version": "2.2.2",
+  "version": "2.2.8",
   "scripts": {
     "start": "node ./bin/www",
     "pretest": "mongodb-runner start",
@@ -37,14 +37,14 @@
     "@tiledesk-ent/tiledesk-server-dialogflow": "^1.1.6",
     "@tiledesk-ent/tiledesk-server-groups": "^1.1.2",
     "@tiledesk-ent/tiledesk-server-jwthistory": "^1.1.9",
-    "@tiledesk-ent/tiledesk-server-mt": "^1.1.6",
+    "@tiledesk-ent/tiledesk-server-mt": "^1.1.7",
     "@tiledesk-ent/tiledesk-server-payments": "^1.1.5",
     "@tiledesk-ent/tiledesk-server-queue": "^1.1.10",
     "@tiledesk-ent/tiledesk-server-request-history": "^1.1.5",
-    "@tiledesk-ent/tiledesk-server-resthook": "^1.1.50",
+    "@tiledesk-ent/tiledesk-server-resthook": "^1.1.51",
     "@tiledesk-ent/tiledesk-server-routing-queue": "^1.1.11",
     "@tiledesk-ent/tiledesk-server-tags": "^1.1.1",
-    "@tiledesk-ent/tiledesk-server-triggers": "^1.1.77",
+    "@tiledesk-ent/tiledesk-server-triggers": "^1.1.78",
     "@tiledesk-ent/tiledesk-server-visitorcounter": "^1.1.1"
   },
   "dependencies": {

package/pubmodules/emailNotification/requestNotification.js

@@ -261,6 +261,7 @@ sendToUserEmailChannelEmail(projectid, message) {
         jwtid: uuidv4()        
       };
 
+
       var recipient = lead.lead_id;
       winston.debug("recipient:"+ recipient);
 
@@ -291,12 +292,13 @@ sendToUserEmailChannelEmail(projectid, message) {
       winston.debug("sourcePage  "+sourcePage);
 
 
-      var tokenQueryString;
-      if(sourcePage && sourcePage.indexOf('?')>-1) { //controllo superfluo ma lascio per indipendenza
-        tokenQueryString =  "&tiledesk_jwt="+encodeURIComponent("JWT "+token)
+      var tokenQueryString;      
+      if(sourcePage && sourcePage.indexOf('?')>-1) {  //controllo superfluo visto che lo metto prima? ma lascio comunque per indipendenza
+        tokenQueryString =  encodeURIComponent("&tiledesk_jwt=JWT "+token)
       }else {
-        tokenQueryString =  "?tiledesk_jwt="+encodeURIComponent("JWT "+token);
+        tokenQueryString =  encodeURIComponent("?tiledesk_jwt=JWT "+token);
       }
+      winston.debug("tokenQueryString:  "+tokenQueryString);
       
       emailService.sendEmailChannelNotification(message.request.lead.email, message, project, tokenQueryString, sourcePage);
     
@@ -610,11 +612,12 @@ sendUserEmail(projectid, message) {
 
                   var tokenQueryString;
                   if(sourcePage && sourcePage.indexOf('?')>-1) {  //controllo superfluo visto che lo metto prima? ma lascio comunque per indipendenza
-                    tokenQueryString =  "&tiledesk_jwt="+encodeURIComponent("JWT "+token)
+                    tokenQueryString =  encodeURIComponent("&tiledesk_jwt=JWT "+token)
                   }else {
-                    tokenQueryString =  "?tiledesk_jwt="+encodeURIComponent("JWT "+token);
+                    tokenQueryString =  encodeURIComponent("?tiledesk_jwt=JWT "+token);
                   }
-
+                  winston.debug("tokenQueryString:  "+tokenQueryString);
+                  
                   emailService.sendNewMessageNotification(lead.email, message, project, tokenQueryString, sourcePage);
               } 
                 
@@ -775,8 +778,27 @@ sendAgentEmail(projectid, savedRequest) {
                             winston.warn("User not found",  savedRequest.participants[0]);
                           } else {
                             winston.verbose("Sending sendNewAssignedRequestNotification to user with email", user.email);
-                            //  if (user.emailverified) {    enable it?     send anyway to improve engagment for new account                
-                              emailService.sendNewAssignedRequestNotification(user.email, savedRequest, project);
+                            //  if (user.emailverified) {    enable it?     send anyway to improve engagment for new account    
+                            
+                            
+                            // var signOptions = {
+                            //   issuer:  'https://tiledesk.com',
+                            //   subject:  'user',
+                            //   audience:  'https://tiledesk.com',
+                            //   jwtid: uuidv4()        
+                            // };
+          
+                            // let userObject = {_id: user._id, firstname: user.firstname, lastname: user.lastname, email: user.email, attributes: user.attributes};
+                            // winston.debug("userObject  ",userObject);
+          
+                  
+                            // var agentToken = jwt.sign(userObject, configSecret, signOptions);
+                            // winston.debug("agentToken  "+agentToken);
+
+                            
+
+
+                            emailService.sendNewAssignedRequestNotification(user.email, savedRequest, project);
                             //  }
                           }
                         });

package/routes/urls.js

@@ -4,7 +4,7 @@ var winston = require('../config/winston');
 
 
 router.get('/redirect', function (req, res) {
-  winston.debug(req.body);
+  winston.debug("redirect: "+ req.query.path);
   res.redirect(req.query.path);
 });
 

package/services/emailService.js

@@ -74,15 +74,15 @@ class EmailService {
     }
     winston.info('EmailService replyEnabled : '+ this.replyEnabled);
 
-    // unused? is empty why?
+    // this is used as fixed reply to url, but this is unused we always return support-group-dynamic
     this.replyTo = process.env.EMAIL_REPLY_TO || config.replyTo;
     winston.info('EmailService replyTo address: '+ this.replyTo);
 
-    this.inboudDomain = process.env.EMAIL_INBOUND_DOMAIN || config.inboundDomain;
-    winston.info('EmailService inboudDomain : '+ this.inboudDomain);
+    this.inboundDomain = process.env.EMAIL_INBOUND_DOMAIN || config.inboundDomain;         
+    winston.info('EmailService inboundDomain : '+ this.inboundDomain);
     
-    this.inboudDomainWithAt = "@"+this.inboudDomain;
-    winston.verbose('EmailService inboudDomainWithAt : '+ this.inboudDomainWithAt);
+    this.inboundDomainDomainWithAt = "@"+this.inboundDomain;
+    winston.verbose('EmailService inboundDomainDomainWithAt : '+ this.inboundDomainDomainWithAt);
 
     this.pass = process.env.EMAIL_PASSWORD;
 
@@ -346,7 +346,11 @@ class EmailService {
   
     let messageId = "notification" + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) { //fai anche per gli altri
+      replyTo = request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (request) { 
       
@@ -356,7 +360,11 @@ class EmailService {
           replyTo = request.attributes.email_replyTo;
         }        
     
-      headers = {"X-TILEDESK-PROJECT-ID": project._id, "X-TILEDESK-REQUEST-ID": request.request_id, "X-TILEDESK-TICKET-ID":request.ticket_id };
+      headers = {
+                  "X-TILEDESK-PROJECT-ID": project._id, 
+                  "X-TILEDESK-REQUEST-ID": request.request_id, 
+                  "X-TILEDESK-TICKET-ID":request.ticket_id,
+                };
 
       winston.verbose("messageId: " + messageId);
       winston.verbose("replyTo: " + replyTo);
@@ -488,7 +496,11 @@ class EmailService {
 
     let messageId = message._id + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = message.request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) {
+      replyTo = message.request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (message.request) { 
       
@@ -507,13 +519,13 @@ class EmailService {
 
     let inReplyTo;
     let references;
-    if (message.attributes) {
-      if (message.attributes.email_messageId) {
-        inReplyTo = message.attributes.email_messageId;
-        }
-        if (message.attributes.email_references) {
-        references = message.attributes.email_references;
-        }
+    if (message.request.attributes) {
+      if (message.request.attributes.email_messageId) {
+        inReplyTo = message.request.attributes.email_messageId;
+      }
+      if (message.request.attributes.email_references) {
+        references = message.request.attributes.email_references;
+      }
     }
     winston.verbose("sendNewAssignedAgentMessageEmailNotification email inReplyTo: "+ inReplyTo);
     winston.verbose("sendNewAssignedAgentMessageEmailNotification email references: "+ references);
@@ -629,7 +641,11 @@ class EmailService {
     
     let messageId = "notification-pooled" + new Date().getTime() + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) {
+      replyTo = request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (request) { 
       
@@ -765,7 +781,11 @@ class EmailService {
 
     let messageId = message._id + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = message.request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) {
+      replyTo = message.request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (message.request) { 
       
@@ -784,13 +804,13 @@ class EmailService {
 
     let inReplyTo;
     let references;
-    if (message.attributes) {
-      if (message.attributes.email_messageId) {
-        inReplyTo = message.attributes.email_messageId;
-        }
-        if (message.attributes.email_references) {
-        references = message.attributes.email_references;
-        }
+    if (message.request.attributes) {
+      if (message.request.attributes.email_messageId) {
+        inReplyTo = message.request.attributes.email_messageId;
+      }
+      if (message.request.attributes.email_references) {
+        references = message.request.attributes.email_references;
+      }
     }
     winston.verbose("sendNewPooledMessageEmailNotification email inReplyTo: "+ inReplyTo);
     winston.verbose("sendNewPooledMessageEmailNotification email references: "+ references);
@@ -904,7 +924,11 @@ class EmailService {
 
     let messageId = message._id + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = message.request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) {
+      replyTo = message.request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (message.request) { 
       
@@ -923,13 +947,13 @@ class EmailService {
 
     let inReplyTo;
     let references;
-    if (message.attributes) {
-      if (message.attributes.email_messageId) {
-        inReplyTo = message.attributes.email_messageId;
-        }
-        if (message.attributes.email_references) {
-        references = message.attributes.email_references;
-        }
+    if (message.request.attributes) {
+      if (message.request.attributes.email_messageId) {
+        inReplyTo = message.request.attributes.email_messageId;
+      }
+      if (message.attributes.email_references) {
+        references = message.request.attributes.email_references;
+      }
     }
     winston.verbose("email inReplyTo: "+ inReplyTo);
     winston.verbose("email references: "+ references);
@@ -1015,6 +1039,8 @@ class EmailService {
     }
     
     winston.debug("msgText: " + msgText);
+    winston.debug("baseScope: " + JSON.stringify(baseScope));
+    
 
     var replacements = {        
       message: message,
@@ -1033,7 +1059,11 @@ class EmailService {
     
     let messageId = message._id + "@" + MESSAGE_ID_DOMAIN;
 
-    let replyTo = message.request.request_id + this.inboudDomainWithAt;
+    let replyTo;
+    if (this.replyEnabled) {
+      replyTo = message.request.request_id + this.inboundDomainDomainWithAt;
+    }
+
     let headers;
     if (message.request) { 
       
@@ -1056,24 +1086,24 @@ class EmailService {
     let cc;
     let ccString;
 
-    if (message.attributes) {
+    if (message.request && message.request.attributes) {
+      winston.debug("email message.request.attributes: ", message.request.attributes);
 
-      winston.verbose("email message.attributes: ", message.attributes);
-      // per email touching manca
-        if (message.attributes.email_messageId) {
-          inReplyTo = message.attributes.email_messageId;
-        }
-        if (message.attributes.email_references) {
-          references = message.attributes.email_references;
-        }
-        if (message.attributes.email_cc) {
-          cc = message.attributes.email_cc;       
-        }
-        winston.verbose("email message.attributes.email_ccStr: "+ message.attributes.email_ccStr);
-        if (message.attributes.email_ccStr!=undefined) {
-          ccString = message.attributes.email_ccStr;
-          winston.verbose("email set ccString");
-        }
+      if (message.request.attributes.email_messageId) {
+        inReplyTo = message.request.attributes.email_messageId;
+      }
+      if (message.request.attributes.email_references) {
+        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");
+      }
     }
     winston.verbose("email inReplyTo: "+ inReplyTo);
     winston.verbose("email references: "+ references);
@@ -1187,7 +1217,7 @@ class EmailService {
 
     var that = this;
 
-    var html = await this.readTemplate('resetPassword.html', project.settings);
+    var html = await this.readTemplate('resetPassword.html');
 
 
     var envTemplate = process.env.EMAIL_RESET_PASSWORD_HTML_TEMPLATE;
@@ -1269,7 +1299,7 @@ class EmailService {
 
     var that = this;
 
-    var html = await this.readTemplateFile('passwordChanged.html');
+    var html = await this.readTemplateFile('beenInvitedExistingUser.html');
 
     var envTemplate = process.env.EMAIL_EXUSER_INVITED_HTML_TEMPLATE;
       winston.debug("envTemplate: " + envTemplate);
@@ -1315,7 +1345,7 @@ class EmailService {
    
     var that = this;
 
-    var html = await this.readTemplateFile('passwordChanged.html');
+    var html = await this.readTemplateFile('beenInvitedNewUser.html');
 
     var envTemplate = process.env.EMAIL_NEWUSER_INVITED_HTML_TEMPLATE;
       winston.debug("envTemplate: " + envTemplate);

package/services/faqService.js

@@ -62,13 +62,12 @@ class FaqService {
         { 'question': '👨🏻‍🦰 I want an agent', 'answer': 'We are looking for an operator.. '+ActionsConstants.CHAT_ACTION_MESSAGE.AGENT, 'intent_display_name': 'agent_handoff', 'topic': 'internal' },
         { 'question': 'Close\nResolved', 'answer': ActionsConstants.CHAT_ACTION_MESSAGE.CLOSE, 'topic': 'internal' }, 
         { 'question': '\\start', 'answer': 'Hello 👋. I\'m a bot 🤖.\n\nChoose one of the options below or write a message to reach our staff.\n* Who are you?\n* Where are you?\n* What can you do?\n* 👨🏻‍🦰 I want an agent', 'intent_display_name': 'start', 'topic': 'internal' },            
-        // { 'question': 'Menu', 'answer': 'Choose one of the options below or write a message to reach our staff.\n* Who are you?\n* Where are you?\n* What can you do?\n* 👨🏻‍🦰 I want an agent','topic': 'internal' },            
         { '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', 'topic': 'internal' }, //TODO se metto spazio n * nn va      
         { 'question': 'What can you do?', 'answer': 'Using natural language processing, I\'m able to find the best answer for your users. I also support images, videos etc.. Let\'s try:\n* Sample Image\n* Sample Video\n* Sample Action tdAction:action1\n* Sample Frame\n* Back to start tdAction:start', 'topic': 'sample' },    
         { 'question': 'Sample Image', 'answer': 'tdImage:https://tiledesk.com/tiledesk-logo-x1.png\n* What can you do?\n* Back to start tdAction:start', 'topic': 'sample' },    
         { 'question': 'Sample Frame', 'answer': 'tdFrame:https://www.emanueleferonato.com/wp-content/uploads/2019/02/runner/\n* What can you do?\n* Back to start tdAction:start', 'topic': 'sample' },    
         { 'question': 'Sample Video', 'answer': 'tdVideo:https://www.youtube.com/embed/EngW7tLk6R8\n* What can you do?\n* Back to start tdAction:start', 'topic': 'sample' },    
-        { 'question': 'Where are you', 'answer': 'We are here ❤️\ntdFrame:https://www.google.com/maps/embed?pb=!1m18!1m12!1m3!1d6087916.923447935!2d8.234804542117423!3d41.836572992140624!2m3!1f0!2f0!3f0!3m2!1i1024!2i768!4f13.1!3m3!1m2!1s0x12d4fe82448dd203%3A0xe22cf55c24635e6f!2sItaly!5e0!3m2!1sen!2sit!4v1613657475377!5m2!1sen!2sit\n* Back to start tdAction:start', 'topic': 'sample' },    
+        { 'question': 'Where are you?', 'answer': 'We are here ❤️\ntdFrame:https://www.google.com/maps/embed?pb=!1m18!1m12!1m3!1d6087916.923447935!2d8.234804542117423!3d41.836572992140624!2m3!1f0!2f0!3f0!3m2!1i1024!2i768!4f13.1!3m3!1m2!1s0x12d4fe82448dd203%3A0xe22cf55c24635e6f!2sItaly!5e0!3m2!1sen!2sit!4v1613657475377!5m2!1sen!2sit\n* Back to start tdAction:start', 'topic': 'sample' },    
 
         // { 'question': 'Sample Webhook', 'answer': 'tdWebhook:https://tiledesk-bot-webhook.tiledesk.repl.co', 'topic': 'sample' },    
         { 'question': 'Sample Action', 'answer': 'Hello 👋 Would you like to take a closer look at our offer?\n* Yes, please tdAction:yes_action\n* No tdAction:no_action','intent_display_name': 'action1', 'topic': 'sample' },    

package/template/email/ticket.html

@@ -132,8 +132,7 @@
                 {{#ifEquals this.type "url"}}
                   <li><a href="{{this.link}}" class="dynamic_button">{{this.value}}</a></li>
                 {{else}}
-                  <li><a href="mailto:{{../message.request.request_id}}@{{baseScope.inboundDomain}}?subject=Re:%20{{../message.request.subject}}&body={{this.value}}" class="dynamic_button">{{this.value}}</a></li>   
-                  <!-- <li><a href="https://api.tiledesk.com/v2/{{project.id}}/link?secret_token={{tokenQueryString}}" class="dynamic_button">{{this.value}}</a></li>     -->
+                  <li><a href="mailto:{{../message.request.request_id}}@{{../baseScope.inboundDomain}}?subject=Re:%20{{../message.request.subject}}&body={{this.value}}" class="dynamic_button">{{this.value}}</a></li>   
                 {{/ifEquals}}
             {{/each}}
 
@@ -142,14 +141,14 @@
     {{/if}}
 {{/if}}
 
-{{#if message.attributes.intent_info}}
+<!-- {{#if message.attributes.intent_info}}
   {{#if message.attributes.intent_info.others}}
     Others:
     {{#each message.attributes.intent_info.others}}
       <li><span>{{this.answer}}</span></li>
     {{/each}}
   {{/if}}
-{{/if}}
+{{/if}} -->
 
 
 

package/test/requestRoute.js

@@ -205,7 +205,6 @@ it('createUpperCaseEmail', function (done) {
 });
 // mocha test/requestRoute.js  --grep 'getbyidWithPartecipatingBots'
 
-
 it('getbyidWithPartecipatingBots', function (done) {
   // this.timeout(10000);