presidium 4.0.0 → 4.0.1

package/DynamoDBGlobalSecondaryIndex.js

@@ -24,6 +24,7 @@ const DynamoDBAttributeValueJSON =
   require('./internal/DynamoDBAttributeValueJSON')
 const hashJSON = require('./internal/hashJSON')
 const sleep = require('./internal/sleep')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 const createExpressionAttributeNames =
   require('./internal/createExpressionAttributeNames')
 const createExpressionAttributeValues =
@@ -227,7 +228,7 @@ class DynamoDBGlobalSecondaryIndex {
       }
     })
 
-    return this.http[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.http, method, url, { headers, body: payload })
   }
 
   /**

package/DynamoDBStream.js

@@ -16,6 +16,7 @@ const AwsAuthorization = require('./internal/AwsAuthorization')
 const AmzDate = require('./internal/AmzDate')
 const Readable = require('./Readable')
 const sleep = require('./internal/sleep')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 const dynamoDBStreamGetStreamsIterator =
   require('./internal/dynamoDBStreamGetStreamsIterator')
 const dynamoDBStreamGetShardsIterator =
@@ -238,7 +239,7 @@ class DynamoDBStream {
       }
     })
 
-    return this.http[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.http, method, url, { headers, body: payload })
   }
 
   /**
@@ -296,7 +297,7 @@ class DynamoDBStream {
       }
     })
 
-    return this.streamsHttp[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.streamsHttp, method, url, { headers, body: payload })
   }
 
   /**

package/DynamoDBTable.js

@@ -25,6 +25,7 @@ const AwsError = require('./internal/AwsError')
 const hashJSON = require('./internal/hashJSON')
 const sleep = require('./internal/sleep')
 const join = require('./internal/join')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 const createExpressionAttributeNames =
   require('./internal/createExpressionAttributeNames')
 const createExpressionAttributeValues =
@@ -216,7 +217,7 @@ class DynamoDBTable {
       }
     })
 
-    return this.http[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.http, method, url, { headers, body: payload })
   }
 
   /**

package/ECR.js

@@ -13,6 +13,7 @@ const AwsAuthorization = require('./internal/AwsAuthorization')
 const AwsError = require('./internal/AwsError')
 const userAgent = require('./userAgent')
 const Readable = require('./Readable')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 
 /**
  * @name ECR
@@ -112,7 +113,7 @@ class ECR {
       }
     })
 
-    return this.http[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.http, method, url, { headers, body: payload })
   }
 
   /**

package/HTTP.js

@@ -141,7 +141,7 @@ class HTTP {
         resolve(response)
       })
 
-      /* TODO remove this an closeConnections from codebase (breaking)
+      /* TODO remove this and closeConnections from codebase (breaking)
       request.on('socket', socket => {
         this._sockets.add(socket)
       })

package/S3Bucket.js

@@ -14,6 +14,7 @@ const AwsAuthorization = require('./internal/AwsAuthorization')
 const AmzDate = require('./internal/AmzDate')
 const AwsError = require('./internal/AwsError')
 const parseURL = require('./internal/parseURL')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 const createS3DeleteObjectError = require('./internal/createS3DeleteObjectError')
 const XML = require('./XML')
 const HTMLEntities = require('html-entities')
@@ -255,7 +256,7 @@ class S3Bucket {
       headers: authorizationHeaders,
     })
 
-    return this.http0[method](url, { headers, body })
+    return retryHTTPRequest(this.http0, method, url, { headers, body })
   }
 
   /**
@@ -319,7 +320,7 @@ class S3Bucket {
       headers: authorizationHeaders,
     })
 
-    return this.http1[method](url, { headers, body })
+    return retryHTTPRequest(this.http1, method, url, { headers, body })
   }
 
   // _putBucketEncryption() -> Promise<>

package/SecretsManager.js

@@ -15,6 +15,7 @@ const userAgent = require('./userAgent')
 const Readable = require('./Readable')
 const handleAwsResponse = require('./internal/handleAwsResponse')
 const retryableErrorNames = require('./internal/retryableErrorNames')
+const retryHTTPRequest = require('./internal/retryHTTPRequest')
 
 /**
  * @name SecretsManager
@@ -114,7 +115,7 @@ class SecretsManager {
       }
     })
 
-    return this.http[method](url, { headers, body: payload })
+    return retryHTTPRequest(this.http, method, url, { headers, body: payload })
   }
 
   /**

package/internal/dynamoDBStreamListStreams.js

@@ -1,5 +1,6 @@
 const Readable = require('../Readable')
 const AwsError = require('./AwsError')
+const sleep = require('./sleep')
 
 /**
  * @name dynamoDBStreamListStreams

package/internal/retryHTTPRequest.js

@@ -0,0 +1,17 @@
+const sleep = require('./sleep')
+
+// retryHTTPRequest(http HTTP, method string, url string, options {
+//   headers: Object<string>,
+//   body: string,
+// }) -> http.ServerResponse
+async function retryHTTPRequest(http, method, url, options) {
+  try {
+    const response = await http[method](url, options)
+    return response
+  } catch {
+    await sleep(1000)
+    return retryHTTPRequest(http, method, url, options)
+  }
+}
+
+module.exports = retryHTTPRequest

package/internal/retryHTTPRequest.test.js

@@ -0,0 +1,40 @@
+const Test = require('thunk-test')
+const assert = require('assert')
+const retryHTTPRequest = require('./retryHTTPRequest')
+
+const test = new Test('retryHTTPRequest', async function integration() {
+  const http1 = {
+    async GET(url, options) {
+      return 'ok'
+    }
+  }
+
+  {
+    const response = await retryHTTPRequest(http1, 'GET', '/test', {})
+    assert.equal(response, 'ok')
+  }
+
+  let retries = 0
+  const http2 = {
+    async GET(url, options) {
+      if (retries === 0) {
+        retries += 1
+        throw new Error('test')
+      }
+      return 'ok'
+    }
+  }
+
+  {
+    const response = await retryHTTPRequest(http2, 'GET', '/test', {})
+    assert.equal(response, 'ok')
+    assert.strictEqual(retries, 1)
+  }
+
+}).case()
+
+if (process.argv[1] == __filename) {
+  test()
+}
+
+module.exports = test

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "presidium",
-  "version": "4.0.0",
+  "version": "4.0.1",
   "description": "A library for creating web services",
   "author": "Richard Tong",
   "license": "CFOSS",