@veryfront/ext-cache-redis 0.1.1288-rc.25587 → 0.1.1288-rc.25610

package/README.md

@@ -76,6 +76,14 @@ ASCII characters and cannot contain Redis glob metacharacters (`*`, `?`, `[`,
 `has(key)`, `stats()`, and `close()`. Entry expiry is carried in
 `entry.expiresAt`. The proxy uses the contract for OAuth service-token caching.
 
+When a SET receives the exact Redis replica-write refusal, the store disposes
+the old client and retries that write once through a fresh connection to the
+configured writer URL. The retry uses the same entry and its remaining lifetime;
+it does not mint another token. Other operation errors and a repeated refusal
+fail closed. Concurrent operations share connection startup, and detached client
+errors cannot invalidate a replacement. `close()` disposes the client and is
+final: subsequent operations cannot reopen the store.
+
 ## Capabilities
 
 - **net `*`:** Redis connection. Narrow to a specific host in your own deployment policy if you're not using a wildcard.

package/esm/extensions/ext-cache-redis/src/redis-cache.d.ts

@@ -40,6 +40,8 @@ export declare class RedisTokenCacheStore implements TokenCacheStore {
     private hits;
     private misses;
     private connected;
+    private connecting;
+    private closed;
     constructor(options: RedisTokenCacheStoreOptions, deps?: {
         logger?: RedisCacheLogger;
         clientFactory?: RedisClientFactory;
@@ -52,7 +54,9 @@ export declare class RedisTokenCacheStore implements TokenCacheStore {
     has(key: string): Promise<boolean>;
     stats(): Promise<TokenCacheStats>;
     close(): Promise<void>;
+    /** Detach before destruction so late socket errors cannot affect its successor. */
+    private invalidate;
     private getConnectedClient;
-    private ensureConnected;
+    private connectClient;
 }
 //# sourceMappingURL=redis-cache.d.ts.map

package/esm/extensions/ext-cache-redis/src/redis-cache.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"redis-cache.d.ts","sourceRoot":"","sources":["../../../../src/extensions/ext-cache-redis/src/redis-cache.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAgB,KAAK,eAAe,EAAE,MAAM,OAAO,CAAC;AAC3D,OAAO,KAAK,EAAE,eAAe,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAgDpG,sDAAsD;AACtD,MAAM,WAAW,2BAA2B;IAC1C,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AACD,qEAAqE;AACrE,MAAM,WAAW,gBAAgB;IAC/B,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IACjD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;CAClD;AAkBD,kEAAkE;AAClE,MAAM,MAAM,kBAAkB,GAAG,CAE/B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,KACtB,eAAe,CAAC;AAErB,qBAAa,oBAAqB,YAAW,eAAe;IAC1D,OAAO,CAAC,MAAM,CAAgC;IAC9C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAS;IAChC,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAS;IAC7B,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAS;IACxC,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAU;IAC9B,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAmB;IAC1C,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAqB;IACnD,OAAO,CAAC,IAAI,CAAK;IACjB,OAAO,CAAC,MAAM,CAAK;IACnB,OAAO,CAAC,SAAS,CAAS;gBAGxB,OAAO,EAAE,2BAA2B,EACpC,IAAI,GAAE;QACJ,MAAM,CAAC,EAAE,gBAAgB,CAAC;QAC1B,aAAa,CAAC,EAAE,kBAAkB,CAAC;KAC/B;IAYR,OAAO,CAAC,GAAG;IAIL,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IA8BjD,GAAG,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IAevD,MAAM,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAalC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAsCtB,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAalC,KAAK,IAAI,OAAO,CAAC,eAAe,CAAC;IAgBjC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;YAiBd,kBAAkB;YAQlB,eAAe;CAsC9B"}
+{"version":3,"file":"redis-cache.d.ts","sourceRoot":"","sources":["../../../../src/extensions/ext-cache-redis/src/redis-cache.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,EAAgB,KAAK,eAAe,EAAE,MAAM,OAAO,CAAC;AAC3D,OAAO,KAAK,EAAE,eAAe,EAAE,eAAe,EAAE,eAAe,EAAE,MAAM,4BAA4B,CAAC;AAgDpG,sDAAsD;AACtD,MAAM,WAAW,2BAA2B;IAC1C,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,GAAG,CAAC,EAAE,OAAO,CAAC;IACd,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,QAAQ,CAAC,EAAE,MAAM,CAAC;CACnB;AACD,qEAAqE;AACrE,MAAM,WAAW,gBAAgB;IAC/B,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IACjD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;IAChD,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,GAAG,IAAI,CAAC;CAClD;AAkBD,kEAAkE;AAClE,MAAM,MAAM,kBAAkB,GAAG,CAE/B,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,KACtB,eAAe,CAAC;AAErB,qBAAa,oBAAqB,YAAW,eAAe;IAC1D,OAAO,CAAC,MAAM,CAAgC;IAC9C,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAS;IAChC,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAS;IAC7B,OAAO,CAAC,QAAQ,CAAC,cAAc,CAAS;IACxC,OAAO,CAAC,QAAQ,CAAC,GAAG,CAAU;IAC9B,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,QAAQ,CAAC,CAAS;IACnC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAmB;IAC1C,OAAO,CAAC,QAAQ,CAAC,aAAa,CAAqB;IACnD,OAAO,CAAC,IAAI,CAAK;IACjB,OAAO,CAAC,MAAM,CAAK;IACnB,OAAO,CAAC,SAAS,CAAS;IAC1B,OAAO,CAAC,UAAU,CAAyC;IAC3D,OAAO,CAAC,MAAM,CAAS;gBAGrB,OAAO,EAAE,2BAA2B,EACpC,IAAI,GAAE;QACJ,MAAM,CAAC,EAAE,gBAAgB,CAAC;QAC1B,aAAa,CAAC,EAAE,kBAAkB,CAAC;KAC/B;IAYR,OAAO,CAAC,GAAG;IAIL,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,GAAG,IAAI,CAAC;IA+BjD,GAAG,CAAC,GAAG,EAAE,MAAM,EAAE,KAAK,EAAE,eAAe,GAAG,OAAO,CAAC,IAAI,CAAC;IA4BvD,MAAM,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAclC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAuCtB,GAAG,CAAC,GAAG,EAAE,MAAM,GAAG,OAAO,CAAC,OAAO,CAAC;IAclC,KAAK,IAAI,OAAO,CAAC,eAAe,CAAC;IAiBjC,KAAK,IAAI,OAAO,CAAC,IAAI,CAAC;IAK5B,mFAAmF;IACnF,OAAO,CAAC,UAAU;IAWlB,OAAO,CAAC,kBAAkB;YAYZ,aAAa;CA8C5B"}

package/esm/extensions/ext-cache-redis/src/redis-cache.js

@@ -69,6 +69,8 @@ export class RedisTokenCacheStore {
     hits = 0;
     misses = 0;
     connected = false;
+    connecting = null;
+    closed = false;
     constructor(options, deps = {}) {
         this.url = options.url;
         this.prefix = validatePrefix(options.prefix ?? DEFAULT_PREFIX);
@@ -83,8 +85,9 @@ export class RedisTokenCacheStore {
         return `${this.prefix}${k}`;
     }
     async get(key) {
+        let client = null;
         try {
-            const client = await this.getConnectedClient();
+            client = await this.getConnectedClient();
             const data = await client.get(this.key(key));
             if (!data) {
                 this.misses++;
@@ -103,42 +106,56 @@ export class RedisTokenCacheStore {
             this.logger.error("[RedisCache] Get error", {
                 error: error instanceof Error ? error.message : String(error),
             });
-            this.connected = false;
+            this.invalidate(client);
             this.misses++;
             throw error;
         }
     }
     async set(key, entry) {
-        try {
-            const client = await this.getConnectedClient();
-            const ttlMs = entry.expiresAt - Date.now();
-            const ttlSeconds = Math.max(1, Math.floor(ttlMs / 1000));
-            await client.setEx(this.key(key), ttlSeconds, JSON.stringify(entry));
-        }
-        catch (error) {
-            this.logger.error("[RedisCache] Set error", {
-                error: error instanceof Error ? error.message : String(error),
-            });
-            this.connected = false;
-            throw error;
+        // Reuse the already minted token: only an explicit replica refusal proves
+        // that retrying this SET cannot duplicate an ambiguously accepted write.
+        const value = JSON.stringify(entry);
+        const expiresAt = entry.expiresAt;
+        for (let attempt = 0; attempt < 2; attempt++) {
+            let client = null;
+            try {
+                client = await this.getConnectedClient();
+                const ttlMs = expiresAt - Date.now();
+                if (attempt > 0 && ttlMs <= 0) {
+                    throw new Error("Redis token-cache entry expired during replica recovery");
+                }
+                const ttlSeconds = Math.max(1, Math.floor(ttlMs / 1000));
+                await client.setEx(this.key(key), ttlSeconds, value);
+                return;
+            }
+            catch (error) {
+                this.logger.error("[RedisCache] Set error", { error: getErrorMessage(error) });
+                this.invalidate(client);
+                if (attempt === 0 && client !== null && error instanceof Error &&
+                    error.message === "READONLY You can't write against a read only replica.")
+                    continue;
+                throw error;
+            }
         }
     }
     async delete(key) {
+        let client = null;
         try {
-            const client = await this.getConnectedClient();
+            client = await this.getConnectedClient();
             await client.del(this.key(key));
         }
         catch (error) {
             this.logger.error("[RedisCache] Delete error", {
                 error: error instanceof Error ? error.message : String(error),
             });
-            this.connected = false;
+            this.invalidate(client);
             throw error;
         }
     }
     async clear() {
+        let client = null;
         try {
-            const client = await this.getConnectedClient();
+            client = await this.getConnectedClient();
             const pattern = `${this.prefix}*`;
             // redis v5: scan cursor is string-based to prevent Number.MAX_SAFE_INTEGER overflow
             let cursor = "0";
@@ -164,31 +181,33 @@ export class RedisTokenCacheStore {
             this.logger.error("[RedisCache] Clear error", {
                 error: error instanceof Error ? error.message : String(error),
             });
-            this.connected = false;
+            this.invalidate(client);
             throw error;
         }
     }
     async has(key) {
+        let client = null;
         try {
-            const client = await this.getConnectedClient();
+            client = await this.getConnectedClient();
             return (await client.exists(this.key(key))) === 1;
         }
         catch (error) {
             this.logger.error("[RedisCache] Has error", {
                 error: error instanceof Error ? error.message : String(error),
             });
-            this.connected = false;
+            this.invalidate(client);
             throw error;
         }
     }
     async stats() {
         let size = 0;
+        let client = null;
         try {
-            const client = await this.getConnectedClient();
+            client = await this.getConnectedClient();
             size = await client.dbSize();
         }
         catch (error) {
-            this.connected = false;
+            this.invalidate(client);
             this.logger.error("[RedisCache] Stats error", {
                 error: error instanceof Error ? error.message : String(error),
             });
@@ -196,32 +215,38 @@ export class RedisTokenCacheStore {
         return { hits: this.hits, misses: this.misses, size, type: "extension" };
     }
     async close() {
-        const client = this.client;
-        if (!client) {
-            this.connected = false;
+        this.closed = true;
+        this.invalidate(this.client);
+    }
+    /** Detach before destruction so late socket errors cannot affect its successor. */
+    invalidate(client) {
+        if (!client || this.client !== client)
             return;
-        }
+        this.client = null;
+        this.connected = false;
         try {
-            await client.close();
+            client.destroy();
         }
-        catch (_) {
-            // expected: close errors are non-critical
+        catch {
+            // Already-closed clients need no further cleanup.
         }
-        finally {
-            this.client = null;
-            this.connected = false;
-        }
-    }
-    async getConnectedClient() {
-        await this.ensureConnected();
-        if (!this.client) {
-            throw new Error("Redis client not available after connect");
-        }
-        return this.client;
     }
-    async ensureConnected() {
+    getConnectedClient() {
+        if (this.closed)
+            return Promise.reject(new Error("Redis token-cache store is closed"));
         if (this.connected && this.client)
-            return;
+            return Promise.resolve(this.client);
+        if (this.connecting)
+            return this.connecting;
+        const connecting = this.connectClient();
+        this.connecting = connecting;
+        void connecting.finally(() => {
+            if (this.connecting === connecting)
+                this.connecting = null;
+        }).catch(() => { });
+        return connecting;
+    }
+    async connectClient() {
         // deno-lint-ignore no-explicit-any
         const clientOpts = {
             url: this.url,
@@ -241,9 +266,13 @@ export class RedisTokenCacheStore {
         if (this.username)
             clientOpts.username = this.username;
         const client = this.clientFactory(clientOpts);
-        // deno-lint-ignore no-explicit-any
-        client.on?.("error", (err) => {
-            this.connected = false;
+        client.on("error", (err) => {
+            if (this.client !== client)
+                return;
+            // Let connect() own its bounded startup reconnect policy. Once ready,
+            // discard a failed socket so the next operation resolves the writer anew.
+            if (this.connected)
+                this.invalidate(client);
             const error = getErrorMessage(err);
             if (isExpectedDisconnect(err)) {
                 this.logger.info("[RedisCache] Client disconnected", { error });
@@ -252,8 +281,19 @@ export class RedisTokenCacheStore {
             this.logger.error("[RedisCache] Client error", { error });
         });
         this.client = client;
-        await client.connect();
-        this.connected = true;
-        this.logger.info("[RedisCache] Connected");
+        try {
+            await client.connect();
+            if (this.closed)
+                throw new Error("Redis token-cache store is closed");
+            if (this.client !== client)
+                throw new Error("Redis client disconnected during connect");
+            this.connected = true;
+            this.logger.info("[RedisCache] Connected");
+            return client;
+        }
+        catch (error) {
+            this.invalidate(client);
+            throw error;
+        }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@veryfront/ext-cache-redis",
-  "version": "0.1.1288-rc.25587",
+  "version": "0.1.1288-rc.25610",
   "description": "Veryfront first-party extension package for ext-cache-redis",
   "keywords": [
     "veryfront",
@@ -63,7 +63,7 @@
     "redis": "5.11.0"
   },
   "peerDependencies": {
-    "veryfront": "^0.1.1288-rc.25587"
+    "veryfront": "^0.1.1288-rc.25610"
   },
   "type": "module",
   "types": "./esm/extensions/ext-cache-redis/src/index.d.ts",
@@ -73,5 +73,5 @@
     "NOTICE",
     "README.md"
   ],
-  "gitHead": "37961872ad0b80ec56ec17c75e33c8c00b187f8f"
+  "gitHead": "046bd33ed82875bbc85596bf561ac131e7e9913b"
 }