feat: Add retry with backoff to requests

index.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const fs = require('graceful-fs');
-const request = require('request');
+const request = require('./lib/request');
 const path = require('path');
 const os = require('os');
 
@@ -330,18 +330,7 @@ function sendRequest(args) {
   // { options, formData }
   const options = args.options
   const formData = args.formData
-  return new Promise((resolve, reject) => {
-    request.post({
-      url: options.endpoint,
-      formData,
-    }, function (err, res, body) {
-      if (err || res.statusCode !== 200) {
-        reject(err || new Error(`${res.statusMessage} (${res.statusCode}) - ${body}`));
-      } else {
-        resolve(options);
-      }
-    });
-  });
+  return request(options.endpoint, formData).then(() => options)
 }
 
 /**

lib/request.js

@@ -0,0 +1,32 @@
+'use strict'
+
+const request = require('request')
+const MAX_ATTEMPTS = 5
+const RETRY_INTERVAL = process.env.BUGSNAG_RETRY_INTERVAL || 1000
+
+module.exports = (url, data) => {
+  return new Promise((resolve, reject) => {
+    let attempts = 0
+    const maybeRetry = (err) => {
+      attempts++
+      if (err && err.isRetryable && attempts < MAX_ATTEMPTS) return setTimeout(go, RETRY_INTERVAL)
+      return reject(err)
+    }
+    const go = () => send(url, data, resolve, maybeRetry)
+    go()
+  })
+}
+
+const send = (url, formData, onSuccess, onError) => {
+  request.post({ url, formData }, (err, res, body) => {
+    if (err || res.statusCode !== 200) {
+      err = err || new Error(`${res.statusMessage} (${res.statusCode}) - ${body}`)
+      if (res.statusCode < 400 || res.statusCode >= 500) {
+        err.isRetryable = true
+      }
+      onError(err)
+    } else {
+      onSuccess()
+    }
+  })
+}