@friggframework/core 2.0.0--canary.623.ce8e6ae.0 → 2.0.0--canary.623.9b7cdf7.0

package/modules/requester/requester.js

@@ -4,6 +4,7 @@ const { FetchError } = require('../../errors');
 const { get } = require('../../assertions');
 
 const DEFAULT_REQUEST_TIMEOUT_MS = 60_000;
+const MAX_AUTH_RETRIES = 3;
 
 class Requester extends Delegate {
     constructor(params) {
@@ -61,6 +62,11 @@ class Requester extends Delegate {
         return resp.text();
     };
 
+    /**
+     * @param {number} attempt - 0-based count of retries already made for
+     *   this call. Indexes `this.backOff` for the next delay and is passed
+     *   back in on each recursive retry.
+     */
     async _request(url, options, attempt = 0) {
         let encodedUrl = encodeURI(url);
         if (options.query) {
@@ -158,12 +164,13 @@ class Requester extends Delegate {
 
             if (status === 401) {
                 if (!this.isRefreshable) {
-                    // One grace retry before invalidating — a single 401
-                    // isn't proof the credential is bad.
-                    if (this.authGraceRetryCount === 0) {
+                    // Up to MAX_AUTH_RETRIES grace retries before invalidating
+                    // — a 401 alone isn't proof the credential is bad.
+                    if (this.authGraceRetryCount < MAX_AUTH_RETRIES) {
+                        const delay =
+                            this.backOff[this.authGraceRetryCount] * 1000;
                         this.authGraceRetryCount++;
                         clearRequestTimer();
-                        const delay = this.backOff[0] * 1000;
                         await new Promise((resolve) =>
                             setTimeout(resolve, delay)
                         );
@@ -177,7 +184,7 @@ class Requester extends Delegate {
                     );
                 }
 
-                if (this.refreshCount === 0) {
+                if (this.refreshCount < MAX_AUTH_RETRIES) {
                     this.refreshCount++;
                     const refreshSucceeded = await this.refreshAuth();
                     if (refreshSucceeded) {
@@ -191,6 +198,8 @@ class Requester extends Delegate {
                         response
                     );
                 }
+
+                throw await this._invalidateAuth(encodedUrl, options, response);
             }
 
             // If the error wasn't retried, throw. FetchError.create reads

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.623.ce8e6ae.0",
+    "version": "2.0.0--canary.623.9b7cdf7.0",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
@@ -38,9 +38,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.623.ce8e6ae.0",
-        "@friggframework/prettier-config": "2.0.0--canary.623.ce8e6ae.0",
-        "@friggframework/test": "2.0.0--canary.623.ce8e6ae.0",
+        "@friggframework/eslint-config": "2.0.0--canary.623.9b7cdf7.0",
+        "@friggframework/prettier-config": "2.0.0--canary.623.9b7cdf7.0",
+        "@friggframework/test": "2.0.0--canary.623.9b7cdf7.0",
         "@prisma/client": "^6.19.3",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -80,5 +80,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "ce8e6ae7f60121290cc48fd3f36653fd18790204"
+    "gitHead": "9b7cdf7b3bcdcd7746a50c30519d2eebe16db23d"
 }