@rabstack/rab-api 1.10.2 → 1.11.1

package/index.cjs.js

@@ -530,9 +530,9 @@ const DEFAULT_STRATEGY = 'url-query';
             strategy: DEFAULT_STRATEGY
         };
     }
-    var _cache_ttl, _ref, _cache_strategy;
+    var _cache_strategy;
     return {
-        ttl: (_ref = (_cache_ttl = cache.ttl) != null ? _cache_ttl : defaultTtl) != null ? _ref : DEFAULT_TTL,
+        ttl: cache.ttl,
         strategy: (_cache_strategy = cache.strategy) != null ? _cache_strategy : DEFAULT_STRATEGY,
         purge: cache.purge
     };
@@ -552,7 +552,7 @@ const DEFAULT_STRATEGY = 'url-query';
     // url-query: path + sorted query string
     const query = req.query;
     if (query && Object.keys(query).length > 0) {
-        const sortedQuery = Object.keys(query).sort().map((key)=>`${key}=${query[key]}`).join('&');
+        const sortedQuery = Object.keys(query).sort().map((key)=>`${encodeURIComponent(key)}=${encodeURIComponent(query[key])}`).join('&');
         return `${path}?${sortedQuery}`;
     }
     return path;
@@ -620,8 +620,10 @@ const controllerHandler = (controller, config)=>{
                 const cacheKey = buildCacheKey(req, cacheConfig.strategy);
                 const cached = await cacheAdapter.get(cacheKey);
                 if (cached) {
+                    console.log(`[rab-api] Cache hit - serving cached response for: ${cacheKey}`);
                     return res.status(200).json(cached);
                 }
+                console.log(`[rab-api] Cache miss - fetching fresh data for: ${cacheKey}`);
             }
             const response = await controller.handler(_extends({}, req, {
                 query,
@@ -639,6 +641,7 @@ const controllerHandler = (controller, config)=>{
             // Cache the response after successful execution (GET only)
             if (isCacheEnabled && isGetRequest) {
                 const cacheKey = buildCacheKey(req, cacheConfig.strategy);
+                console.log(`[rab-api] Caching response for ${cacheConfig.ttl}s: ${cacheKey}`);
                 // Don't await - cache in background
                 cacheAdapter.set(cacheKey, jsonResponse, cacheConfig.ttl).catch(()=>{
                 // Silently ignore cache errors
@@ -647,6 +650,7 @@ const controllerHandler = (controller, config)=>{
             // Handle purge after successful mutation
             if (hasPurge && cacheAdapter) {
                 const keys = resolvePurgeKeys(cacheConfig.purge, req);
+                console.log(`[rab-api] Purging cache keys: ${keys.join(', ')}`);
                 // Purge in background no await for promises you could use setImmediate if needed
                 Promise.all(keys.map((key)=>cacheAdapter.del(key))).catch(()=>{
                 // Silently ignore purge errors

package/index.esm.js

@@ -528,9 +528,9 @@ const DEFAULT_STRATEGY = 'url-query';
             strategy: DEFAULT_STRATEGY
         };
     }
-    var _cache_ttl, _ref, _cache_strategy;
+    var _cache_strategy;
     return {
-        ttl: (_ref = (_cache_ttl = cache.ttl) != null ? _cache_ttl : defaultTtl) != null ? _ref : DEFAULT_TTL,
+        ttl: cache.ttl,
         strategy: (_cache_strategy = cache.strategy) != null ? _cache_strategy : DEFAULT_STRATEGY,
         purge: cache.purge
     };
@@ -550,7 +550,7 @@ const DEFAULT_STRATEGY = 'url-query';
     // url-query: path + sorted query string
     const query = req.query;
     if (query && Object.keys(query).length > 0) {
-        const sortedQuery = Object.keys(query).sort().map((key)=>`${key}=${query[key]}`).join('&');
+        const sortedQuery = Object.keys(query).sort().map((key)=>`${encodeURIComponent(key)}=${encodeURIComponent(query[key])}`).join('&');
         return `${path}?${sortedQuery}`;
     }
     return path;
@@ -618,8 +618,10 @@ const controllerHandler = (controller, config)=>{
                 const cacheKey = buildCacheKey(req, cacheConfig.strategy);
                 const cached = await cacheAdapter.get(cacheKey);
                 if (cached) {
+                    console.log(`[rab-api] Cache hit - serving cached response for: ${cacheKey}`);
                     return res.status(200).json(cached);
                 }
+                console.log(`[rab-api] Cache miss - fetching fresh data for: ${cacheKey}`);
             }
             const response = await controller.handler(_extends({}, req, {
                 query,
@@ -637,6 +639,7 @@ const controllerHandler = (controller, config)=>{
             // Cache the response after successful execution (GET only)
             if (isCacheEnabled && isGetRequest) {
                 const cacheKey = buildCacheKey(req, cacheConfig.strategy);
+                console.log(`[rab-api] Caching response for ${cacheConfig.ttl}s: ${cacheKey}`);
                 // Don't await - cache in background
                 cacheAdapter.set(cacheKey, jsonResponse, cacheConfig.ttl).catch(()=>{
                 // Silently ignore cache errors
@@ -645,6 +648,7 @@ const controllerHandler = (controller, config)=>{
             // Handle purge after successful mutation
             if (hasPurge && cacheAdapter) {
                 const keys = resolvePurgeKeys(cacheConfig.purge, req);
+                console.log(`[rab-api] Purging cache keys: ${keys.join(', ')}`);
                 // Purge in background no await for promises you could use setImmediate if needed
                 Promise.all(keys.map((key)=>cacheAdapter.del(key))).catch(()=>{
                 // Silently ignore purge errors

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rabstack/rab-api",
-  "version": "1.10.2",
+  "version": "1.11.1",
   "description": "A TypeScript REST API framework built on Express.js with decorator-based routing, dependency injection, and built-in validation",
   "author": "Softin",
   "license": "MIT",
@@ -19,7 +19,7 @@
   },
   "peerDependencies": {
     "express": "^5.1.0",
-    "typedi": "^0.10.0",
+    "typedi": "0.10.0",
     "jsonwebtoken": "^9.0.0",
     "compose-middleware": "^5.0.0",
     "reflect-metadata": "^0.2.2"