Fixes to address changed interface for async.whilst in async 3.0.1

awslabs · Jan 22, 2020 · 583f523 · 583f523
1 parent 8fa11e3
commit 583f523
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/common.js b/common.js
@@ -172,9 +172,9 @@ exports.retryableUpdate = function (dynamoDB, updateRequest, callback) {
     var writeRetryLimit = 100;
     var done = false;
 
-    async.whilst(function () {
+    async.whilst(function test(test_cb) {
         // retry until the try count is hit
-        return tryNumber < writeRetryLimit && done === false;
+        test_cb(null, tryNumber < writeRetryLimit && done === false);
     }, function (asyncCallback) {
         tryNumber++;
 

diff --git a/dist/AWSLambdaRedshiftLoader-2.7.2.zip b/dist/AWSLambdaRedshiftLoader-2.7.2.zip
diff --git a/index.js b/index.js
@@ -95,10 +95,10 @@ exports.getConfigWithRetry = function (prefix, callback) {
         ConsistentRead: true
     };
 
-    async.whilst(function () {
+    async.whilst(function test(test_cb) {
         // return OK if the proceed flag has been set, or if
         // we've hit the retry count
-        return !proceed && tryNumber < lookupConfigTries;
+        test_cb(!proceed && tryNumber < lookupConfigTries);
     }, function (callback) {
         tryNumber++;
 

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
     "name": "aws-lambda-redshift-loader",
     "description": "An Amazon Redshift Database Loader written for AWS Lambda",
-    "version": "2.7.1",
+    "version": "2.7.2",
     "homepage": "http://github.com/awslabs/aws-lambda-redshift-loader",
     "bugs": {
         "url": "http://github.com/awslabs/aws-lambda-redshift-loader/issues",