playov2-js-utilities 0.3.37 → 0.3.39

package/lib/taskscheduler/index.js

@@ -16,8 +16,9 @@ const MAX_RETRY_COUNT = 5;
  * @param {String} name - Optional - If "RetryableException", then function will be retried till MAX_RETRY_COUNT
  */
 function CTException(message, name) {
-    this.message = message;
-    this.name = name;
+    const error = new Error(message);
+    error.name = name;
+    return error;
 };
 
 /**
@@ -38,8 +39,11 @@ const saveFutureTask = async (queueId, taskId, payload, eta, count = MAX_RETRY_C
             throw CTException('eta can not be before 30 days from now. Schedule directly!', 'ValidationException')
         }
 
+        const { url, httpMethod } = payload;
         const taskData = {
             taskId,
+            url,
+            httpMethod,
             payload,
             queueId,
             activationTime: eta,
@@ -52,10 +56,15 @@ const saveFutureTask = async (queueId, taskId, payload, eta, count = MAX_RETRY_C
         await task.save();
 
     } catch (err) {
-        if (err.message !== "RetryableException") {
+        const errMessage = err && err.message ? err.message : 'Exception';
+        // in case there's an exception which shouldn't be retried, such as improper parameter, this throws the error to be 
+        // handled by the originating calling function
+        if (errMessage !== "RetryableException") {
             throw err;
         }
 
+        // This is a mechanism to handle unexpected network failure / disconnection to db.
+        // This seems lame though, probably should pair with set timeout for exponential back off.
         if (count === 0) {
             Logger.prepareAlertLog(requestId, { func: 'saveFutureTask', queueId, taskId, payload, eta, err }, 'saving future task failed!');
             throw err;
@@ -65,7 +74,6 @@ const saveFutureTask = async (queueId, taskId, payload, eta, count = MAX_RETRY_C
     }
 };
 
-
 const deleteScheduledTask = async (queueId, taskId, count = MAX_RETRY_COUNT, requestId) => {
     try {
         if (!queueId || !taskId) {
@@ -92,6 +100,5 @@ const deleteScheduledTask = async (queueId, taskId, count = MAX_RETRY_COUNT, req
 
 module.exports = {
     saveFutureTask,
-    // pushScheduledTaskToQueue,
     deleteScheduledTask
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playov2-js-utilities",
-  "version": "0.3.37",
+  "version": "0.3.39",
   "description": "Private package for JS utility functions",
   "main": "index.js",
   "scripts": {