Update sailor 2 6 19 (#34)

Update Sailor version to 2.6.19

Author: Olha Virolainen

.circleci/config.yml

@@ -7,6 +7,9 @@ jobs:
       - checkout
       - restore_cache:
           key: dependency-cache-{{ checksum "package.json" }}
+      - run:
+          name: Audit Dependencies
+          command: npm audit --audit-level=high
       - run:
           name: Installing Dependencies
           command: npm install

CHANGELOG.md

@@ -1,3 +1,9 @@
+# 1.0.2 (November 27, 2020)
+
+* Update Sailor version to 2.6.19
+* Annual audit of the component code to check if it exposes a sensitive data in the logs
+* Annual npm vulnerabilities audit
+
 # 1.0.1 (July 31, 2020)
 
 * Fix retrieving mail folders issue

component.json

@@ -44,7 +44,7 @@
           "type": "polling",
           "help": {
             "description": "Contacts",
-            "link": "/components/outlook/triggers#contacts"
+            "link": "/components/outlook/#contacts"
           },
           "metadata": {
             "in": {},
@@ -56,7 +56,7 @@
           "main": "./lib/triggers/readMail.js",
           "help": {
             "description": "Poll for New Mail",
-            "link": "/components/outlook/triggers#poll-for-new-mail"
+            "link": "/components/outlook/#poll-for-new-mail"
           },
           "type": "polling",
           "fields": {
@@ -101,7 +101,7 @@
           "main": "./lib/actions/checkAvailability.js",
           "help": {
             "description": "Action for checking availability",
-            "link": "/components/outlook/actions#check-availability"
+            "link": "/components/outlook/#check-availability"
           },
           "metadata": {
             "in": "./lib/schemas/checkAvailability.in.json",
@@ -113,7 +113,7 @@
           "main": "./lib/actions/findNextAvailableTime.js",
           "help": {
             "description": "Action for calculating next availability time",
-            "link": "/components/outlook/actions#find-next-available-time"
+            "link": "/components/outlook/#find-next-available-time"
           },
           "metadata": {
             "in": "./lib/schemas/findNextAvailableTime.in.json",
@@ -125,7 +125,7 @@
           "main": "./lib/actions/createEvent.js",
           "help": {
             "description": "Action for creating new event",
-            "link": "/components/outlook/actions#create-event"
+            "link": "/components/outlook/#create-event"
           },
           "fields": {
             "calendarId": {
@@ -304,7 +304,7 @@
           "main": "./lib/actions/moveMail.js",
           "help": {
             "description": "Action for moving message between folders or soft-deleting message if the `Destination Mail Folder` folder isn't specified",
-            "link": "/components/outlook/actions#move-mail"
+            "link": "/components/outlook/#move-mail"
           },
           "fields": {
             "originalMailFolders": {

lib/actions/createEvent.js

@@ -7,16 +7,15 @@ const { Client } = require('../Client');
 exports.process = async function process(msg, cfg) {
   const client = new Client(this, cfg);
   const { calendarId } = cfg;
-  this.logger.info(`About to create event in calendar with id=${calendarId}`);
+  this.logger.info('About to create event in specified calendar...');
   const postRequestBody = await processEventData(this.logger, cfg, msg.body);
-  this.logger.trace('Created postRequestBody: %j', postRequestBody);
+  this.logger.debug('postRequestBody is created');
   try {
     const data = await client.createEvent(calendarId, postRequestBody);
-    this.logger.info(`Successfully created event with ID=${data.id}`);
+    this.logger.info('Successfully created event');
     const messageBody = _.omitBy(data, (value, key) => key.startsWith('@odata.'));
     messageBody.calendarId = cfg.calendarId;
     this.logger.info('Emitting data ...');
-    this.logger.trace('Emitting data: %j', messageBody);
     await this.emit('data', messages.newMessageWithBody(messageBody));
   } catch (e) {
     this.logger.error('Oops! Error occurred');
@@ -29,7 +28,7 @@ exports.getCalendars = async function getCalendars(cfg) {
   const client = new Client(this, cfg);
   this.logger.info('Getting calendar data ...');
   const items = await client.getMyCalendars();
-  this.logger.trace('Found calendars: %j', items);
+  this.logger.debug('Found %s calendar(s)', items.length);
   this.logger.info('Processing calendar data ...');
   const result = {};
   items.forEach((item) => {

lib/actions/moveMail.js

@@ -13,7 +13,7 @@ exports.process = async function process(msg, cfg) {
     currentFolder = await client.getMailFolderById(destinationId);
   } else {
     const deleteditemsFolder = await client.getDeleteditemsFolder();
-    this.logger.trace('Found deleteditemsFolder: %j', deleteditemsFolder);
+    this.logger.trace('deleteditemsFolder found');
     currentFolder = deleteditemsFolder;
     destinationId = deleteditemsFolder.id;
   }

lib/triggers/contacts.js

@@ -11,14 +11,13 @@ const { Client } = require('../Client');
  * @param snapshot - snapshot that stores the data between the runs
  */
 exports.process = async function process(msg, cfg, snapshot) {
-  this.logger.info('Snapshot is %j', snapshot);
+  this.logger.info('Contacts trigger starting...');
 
   const client = new Client(this, cfg);
   // Should be in ISO-Date format
   snapshot.lastModifiedDateTime = snapshot.lastModifiedDateTime || new Date(0).toISOString();
   this.logger.info('Selecting contacts that was modified since %s', snapshot.lastModifiedDateTime);
   const result = await client.getMyLatestContacts(snapshot.lastModifiedDateTime);
-  this.logger.trace('Result %j', result);
   this.logger.info('Found %s contacts', result.length);
   if (result.length > 0) {
     result.forEach((elem) => {

lib/triggers/readMail.js

@@ -4,25 +4,23 @@ const { Client } = require('../Client');
 const { getFolders } = require('../utils/selectViewModels');
 
 exports.process = async function process(msg, cfg, snapshot) {
-  this.logger.trace('Incoming configuration: %j', cfg);
-  this.logger.trace('Incoming message: %j', msg);
-  this.logger.trace('Incoming snapshot: %j', snapshot);
+  this.logger.info('Poll for New Mail trigger starting...');
   const {
     folderId, startTime, pollOnlyUnreadMail, emitBehavior = 'emitIndividually',
   } = cfg;
   snapshot.lastModifiedDateTime = snapshot.lastModifiedDateTime || startTime || new Date(0).toISOString();
-  this.logger.info(`Getting message from folder with id: ${folderId} starting from: ${snapshot.lastModifiedDateTime}, pollOnlyUnreadMail ${pollOnlyUnreadMail || false}`);
+  this.logger.info(`Getting message from specified folder, starting from: ${snapshot.lastModifiedDateTime}, pollOnlyUnreadMail ${pollOnlyUnreadMail || false}`);
   const client = new Client(this, cfg);
   const results = await client.getLatestMessages(folderId, snapshot.lastModifiedDateTime, pollOnlyUnreadMail);
-  this.logger.trace('Results: %j', results);
+  this.logger.debug('Results are received');
   if (results.length > 0) {
     this.logger.info(`New Mails found, going to emit, Emit Behavior: ${emitBehavior}...`);
     switch (emitBehavior) {
       case 'emitIndividually':
         this.logger.info('Starting to emit individually found mails...');
         // eslint-disable-next-line no-restricted-syntax
         for (const result of results) {
-          this.logger.trace('Emit Mail: %j', result);
+          this.logger.debug('Emit Mail...');
           // eslint-disable-next-line no-await-in-loop
           await this.emit('data', messages.newMessageWithBody(result));
         }

lib/utils/selectViewModels.js

@@ -15,7 +15,6 @@ exports.getFolders = async function getFolders(cfg) {
   const client = new Client(this, cfg);
   this.logger.info('Getting list of folders...');
   const items = await client.getMailFolders();
-  this.logger.trace('Found folders: %j', items);
   this.logger.info('Processing list of folders...');
   const result = {};
   for (const item of items) {
@@ -24,6 +23,6 @@ exports.getFolders = async function getFolders(cfg) {
       await getChildFolders(client, item.id, result);
     }
   }
-  this.logger.trace('Result: %j', result);
+  this.logger.debug('Return result...');
   return result;
 };