Expose default retry logic as defaultOptions.onShouldRetry (#637)

* move default retry logic on to defaultOptions to allow user to reuse it inside custom `onShouldRetry` option

* correct onShouldRetry as it'll never be called with just an error

* remove unused parameters causing build error

* move default on should retry into it's own function.

lib/index.d.ts

@@ -33,7 +33,7 @@ interface UploadOptions {
   onSuccess?: (() => void) | null
   onError?: ((error: Error | DetailedError) => void) | null
   onShouldRetry?:
-    | ((error: Error | DetailedError, retryAttempt: number, options: UploadOptions) => boolean)
+    | ((error: DetailedError, retryAttempt: number, options: UploadOptions) => boolean)
     | null
   onUploadUrlAvailable?: (() => void) | null
 

lib/upload.js

@@ -23,7 +23,7 @@ const defaultOptions = {
   addRequestId: false,
   onBeforeRequest: null,
   onAfterResponse: null,
-  onShouldRetry: null,
+  onShouldRetry: defaultOnShouldRetry,
 
   chunkSize: Infinity,
   retryDelays: [0, 1000, 3000, 5000],
@@ -1000,7 +1000,7 @@ function isOnline() {
 
 /**
  * Checks whether or not it is ok to retry a request.
- * @param {Error} err the error returned from the last request
+ * @param {Error|DetailedError} err the error returned from the last request
  * @param {number} retryAttempt the number of times the request has already been retried
  * @param {object} options tus Upload options
  *
@@ -1026,6 +1026,15 @@ function shouldRetry(err, retryAttempt, options) {
     return options.onShouldRetry(err, retryAttempt, options)
   }
 
+  return defaultOnShouldRetry(err)
+}
+
+/**
+ * determines if the request should be retried. Will only retry if not a status 4xx except a 409 or 423
+ * @param {DetailedError} err
+ * @returns {boolean}
+ */
+function defaultOnShouldRetry(err) {
   const status = err.originalResponse ? err.originalResponse.getStatus() : 0
   return (!inStatusCategory(status, 400) || status === 409 || status === 423) && isOnline()
 }