@percy/client 1.31.9-beta.1 → 1.31.9-beta.3

package/dist/utils.js

@@ -91,7 +91,8 @@ export function retry(fn, {
   interval = 50
 }) {
   return new Promise((resolve, reject) => {
-    let run = () => fn(resolve, reject, retry);
+    const maxRetries = retries;
+    let run = () => fn(resolve, reject, retry, maxRetries - retries);
 
     // wait an interval to try again or reject with the error
     let retry = err => {
@@ -169,7 +170,7 @@ export async function request(url, options = {}, callback) {
     headers,
     port
   });
-  return retry((resolve, reject, retry) => {
+  return retry((resolve, reject, retry, attemptNumber) => {
     let handleError = error => {
       if (handleError.handled) return;
       handleError.handled = true;
@@ -182,8 +183,12 @@ export async function request(url, options = {}, callback) {
         meta.cfRay = response.headers['cf-ray'];
       }
 
-      // maybe retry 404s, always retry 500s, or retry specific errors
-      let shouldRetry = response ? retryNotFound && response.statusCode === 404 || response.statusCode >= 500 && response.statusCode < 600 : !!error.code && RETRY_ERROR_CODES.includes(error.code);
+      // maybe retry 404s, always retry 500s, 409s (conflict), or retry specific errors
+      let shouldRetry = response ? retryNotFound && response.statusCode === 404 || response.statusCode >= 500 && response.statusCode < 600 || response.statusCode === 409 : !!error.code && RETRY_ERROR_CODES.includes(error.code);
+      if (shouldRetry) {
+        let reason = response ? `status ${response.statusCode}` : `error ${error.code}`;
+        logger('client:utils').debug(`Retrying request (attempt ${attemptNumber + 1}) due to ${reason}`);
+      }
       return shouldRetry ? retry(error) : reject(error);
     };
     let handleFinished = async (body, res) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@percy/client",
-  "version": "1.31.9-beta.1",
+  "version": "1.31.9-beta.3",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -33,11 +33,11 @@
     "test:coverage": "yarn test --coverage"
   },
   "dependencies": {
-    "@percy/config": "1.31.9-beta.1",
-    "@percy/env": "1.31.9-beta.1",
-    "@percy/logger": "1.31.9-beta.1",
+    "@percy/config": "1.31.9-beta.3",
+    "@percy/env": "1.31.9-beta.3",
+    "@percy/logger": "1.31.9-beta.3",
     "pac-proxy-agent": "^7.0.2",
     "pako": "^2.1.0"
   },
-  "gitHead": "367764ca26d9da4b422c709ec35c34a0d25e4ab4"
+  "gitHead": "38b33a40a51939de5d35df38b7652840e2bd6d9c"
 }