@genesislcap/foundation-auth 15.68.0 → 15.68.1

package/dist/dts/machine/session-refresh.d.ts

@@ -11,10 +11,10 @@ export declare const DEFAULT_LOCK_WAIT_MS: number;
  * re-checks the session first, so a refresh done by the other tab is picked up without refreshing again.
  *
  * @param options.lockWaitMs - How long to wait for another tab's lock before giving up.
- * @returns true when the session is valid afterwards; false when the server rejected the refresh (NACK or 4xx) or the
- * lock was still held at the deadline.
- * @throws When the refresh could not complete (network error, 5xx, storage unavailable), so callers can tell a
- * transient failure from an expired refresh cookie.
+ * @returns true when the session is valid afterwards; false when the server rejected the refresh (NACK, or a 4xx other
+ * than 408/429) or the lock was still held at the deadline.
+ * @throws When the refresh could not complete (network error, 408, 429, 5xx, storage unavailable), so callers can tell
+ * a transient failure from an expired refresh cookie.
  * @internal
  */
 export declare function refreshCookieSession({ lockWaitMs, }?: {

package/dist/dts/machine/session-refresh.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"session-refresh.d.ts","sourceRoot":"","sources":["../../../src/machine/session-refresh.ts"],"names":[],"mappings":"AAcA;;GAEG;AACH,eAAO,MAAM,oBAAoB,QAA2C,CAAC;AAoC7E;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,oBAAoB,CAAC,EACzC,UAAiC,GAClC,GAAE;IAAE,UAAU,CAAC,EAAE,MAAM,CAAA;CAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAWjD"}
+{"version":3,"file":"session-refresh.d.ts","sourceRoot":"","sources":["../../../src/machine/session-refresh.ts"],"names":[],"mappings":"AAaA;;GAEG;AACH,eAAO,MAAM,oBAAoB,QAA2C,CAAC;AAuB7E;;;;;;;;;;;;;;GAcG;AACH,wBAAsB,oBAAoB,CAAC,EACzC,UAAiC,GAClC,GAAE;IAAE,UAAU,CAAC,EAAE,MAAM,CAAA;CAAO,GAAG,OAAO,CAAC,OAAO,CAAC,CAOjD"}

package/dist/esm/machine/session-refresh.js

@@ -1,5 +1,5 @@
 import { __awaiter } from "tslib";
-import { onOtherTabReauthComplete, REAUTH_LOCK_TIMEOUT_MS, SessionRefreshResult, tryRefreshSession, } from '@genesislcap/foundation-comms';
+import { REAUTH_LOCK_TIMEOUT_MS, SessionRefreshResult, tryRefreshSession, waitForOtherTabReauth, } from '@genesislcap/foundation-comms';
 import { logger } from '../utils';
 /**
  * Upper bound between lock checks, so a lock abandoned without a storage event is still picked up once it goes stale.
@@ -9,29 +9,19 @@ const LOCK_RECHECK_MS = 1000;
  * Long enough for a lock left by another tab to go stale and be cleared by `tryRefreshSession`.
  */
 export const DEFAULT_LOCK_WAIT_MS = REAUTH_LOCK_TIMEOUT_MS + LOCK_RECHECK_MS;
-const HTTP_CLIENT_ERROR_MIN = 400;
-const HTTP_SERVER_ERROR_MIN = 500;
 const isRefreshed = (result) => result === SessionRefreshResult.SUCCESS || result === SessionRefreshResult.ALREADY_VALID;
-const isRejectedByServer = (error) => {
-    const status = error === null || error === void 0 ? void 0 : error.status;
-    return (typeof status === 'number' && status >= HTTP_CLIENT_ERROR_MIN && status < HTTP_SERVER_ERROR_MIN);
+// AbortSignal.timeout's timer doesn't keep Node's event loop alive, so a test waiting only on it exits mid-wait.
+const timeoutSignal = (ms) => {
+    const controller = new AbortController();
+    setTimeout(() => controller.abort(), ms);
+    return controller.signal;
 };
-const waitForLockRelease = (timeoutMs) => new Promise((resolve) => {
-    const timeout = setTimeout(() => {
-        unsubscribe();
-        resolve();
-    }, timeoutMs);
-    const unsubscribe = onOtherTabReauthComplete(() => {
-        clearTimeout(timeout);
-        resolve();
-    });
-});
 const refreshOnceLockIsFree = (deadline) => __awaiter(void 0, void 0, void 0, function* () {
     const result = yield tryRefreshSession({ reloadOnFailure: false });
     if (result !== SessionRefreshResult.HANDLED_BY_OTHER_TAB || Date.now() >= deadline) {
         return result;
     }
-    yield waitForLockRelease(Math.min(LOCK_RECHECK_MS, Math.max(0, deadline - Date.now())));
+    yield waitForOtherTabReauth(timeoutSignal(Math.min(LOCK_RECHECK_MS, Math.max(0, deadline - Date.now()))));
     return refreshOnceLockIsFree(deadline);
 });
 /**
@@ -43,10 +33,10 @@ const refreshOnceLockIsFree = (deadline) => __awaiter(void 0, void 0, void 0, fu
  * re-checks the session first, so a refresh done by the other tab is picked up without refreshing again.
  *
  * @param options.lockWaitMs - How long to wait for another tab's lock before giving up.
- * @returns true when the session is valid afterwards; false when the server rejected the refresh (NACK or 4xx) or the
- * lock was still held at the deadline.
- * @throws When the refresh could not complete (network error, 5xx, storage unavailable), so callers can tell a
- * transient failure from an expired refresh cookie.
+ * @returns true when the session is valid afterwards; false when the server rejected the refresh (NACK, or a 4xx other
+ * than 408/429) or the lock was still held at the deadline.
+ * @throws When the refresh could not complete (network error, 408, 429, 5xx, storage unavailable), so callers can tell
+ * a transient failure from an expired refresh cookie.
  * @internal
  */
 export function refreshCookieSession() {
@@ -55,10 +45,6 @@ export function refreshCookieSession() {
             return isRefreshed(yield refreshOnceLockIsFree(Date.now() + lockWaitMs));
         }
         catch (error) {
-            if (isRejectedByServer(error)) {
-                logger.debug('Session refresh rejected.', error);
-                return false;
-            }
             logger.warn('Session refresh could not complete.', error);
             throw error;
         }

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/foundation-auth",
   "description": "Genesis Foundation Auth",
-  "version": "15.68.0",
+  "version": "15.68.1",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/esm/index.js",
   "types": "dist/foundation-auth.d.ts",
@@ -134,24 +134,24 @@
     }
   },
   "devDependencies": {
-    "@genesislcap/foundation-testing": "15.68.0",
-    "@genesislcap/genx": "15.68.0",
-    "@genesislcap/ts-builder": "15.68.0",
-    "@genesislcap/uvu-playwright-builder": "15.68.0",
-    "@genesislcap/vite-builder": "15.68.0",
-    "@genesislcap/webpack-builder": "15.68.0"
+    "@genesislcap/foundation-testing": "15.68.1",
+    "@genesislcap/genx": "15.68.1",
+    "@genesislcap/ts-builder": "15.68.1",
+    "@genesislcap/uvu-playwright-builder": "15.68.1",
+    "@genesislcap/vite-builder": "15.68.1",
+    "@genesislcap/webpack-builder": "15.68.1"
   },
   "dependencies": {
-    "@genesislcap/foundation-comms": "15.68.0",
-    "@genesislcap/foundation-events": "15.68.0",
-    "@genesislcap/foundation-i18n": "15.68.0",
-    "@genesislcap/foundation-logger": "15.68.0",
-    "@genesislcap/foundation-state-machine": "15.68.0",
-    "@genesislcap/foundation-store": "15.68.0",
-    "@genesislcap/foundation-ui": "15.68.0",
-    "@genesislcap/foundation-user": "15.68.0",
-    "@genesislcap/foundation-utils": "15.68.0",
-    "@genesislcap/foundation-zero": "15.68.0",
+    "@genesislcap/foundation-comms": "15.68.1",
+    "@genesislcap/foundation-events": "15.68.1",
+    "@genesislcap/foundation-i18n": "15.68.1",
+    "@genesislcap/foundation-logger": "15.68.1",
+    "@genesislcap/foundation-state-machine": "15.68.1",
+    "@genesislcap/foundation-store": "15.68.1",
+    "@genesislcap/foundation-ui": "15.68.1",
+    "@genesislcap/foundation-user": "15.68.1",
+    "@genesislcap/foundation-utils": "15.68.1",
+    "@genesislcap/foundation-zero": "15.68.1",
     "@microsoft/fast-components": "2.30.6",
     "@microsoft/fast-element": "1.14.0",
     "@microsoft/fast-foundation": "2.50.0",