adding explicit retry count logic (#119)

Added explicit retry count logic to prevent underlying & misleading
exceptions.

### Testing

Ran large PDFs via integration tests.

Author: jordan-homan

src/hooks/custom/SplitPdfHook.ts

@@ -207,6 +207,7 @@ export class SplitPdfHook
     // We need to hardcode them here until we're able to reuse the SDK
     // from within this hook
 
+    const allowedRetries = 3;
     const retryConfig = {
         strategy: "backoff",
         backoff: {
@@ -219,13 +220,19 @@ export class SplitPdfHook
 
     const retryCodes = ["502", "503", "504"];
 
+
     this.partitionRequests[operationID] = async.parallelLimit(
       requests.map((req, pageIndex) => async () => {
         const pageNumber = pageIndex + startingPageNumber;
+        let retryCount = 0;
         try {
          const response = await retry(
               async () => {
-                return await this.client!.request(req);
+                retryCount++;
+                if (retryCount > allowedRetries) {
+                  throw new Error(`Number of retries exceeded for page ${pageNumber}`);
+                }
+                return await this.client!.request(req.clone());
               },
               { config: retryConfig, statusCodes: retryCodes }
           );

src/hooks/custom/common.ts

@@ -32,8 +32,7 @@ export class HTTPClientExtension extends HTTPClient {
   }
 
   override async request(request: Request): Promise<Response> {
-    const clone = request.clone();
-    if (clone.url === "https://no-op/") {
+    if (request.url === "https://no-op/") {
       return new Response('{}', {
         headers: [
             ["fake-response", "fake-response"]
@@ -42,6 +41,6 @@ export class HTTPClientExtension extends HTTPClient {
         statusText: 'OK_NO_OP'
       });
     }
-    return super.request(clone);
+    return super.request(request);
    }
 }