Add 'cloneData' option to buildMemoryStorage

src/storage/memory.ts

@@ -1,5 +1,5 @@
 import { buildStorage, canStale, isExpired } from './build';
-import type { AxiosStorage, StorageValue } from './types';
+import type { AxiosStorage, NotEmptyStorageValue, StorageValue } from './types';
 
 /**
  * Modern function to natively deep clone.
@@ -31,19 +31,22 @@ declare const structuredClone: (<T>(value: T) => T) | undefined;
  * ```
  *
  * @param {boolean} cloneData If the data returned by `find()` should be cloned to avoid
- *   mutating the original data outside the `set()` method.
+ *   mutating the original data outside the `set()` method. Disabled is default
  * @param {number | false} cleanupInterval The interval in milliseconds to run a
  *   setInterval job of cleaning old entries. If false, the job will not be created.
  *   Disabled is default
  * @param {number | false} maxEntries The maximum number of entries to keep in the
  *   storage. Its hard to determine the size of the entries, so a smart FIFO order is used
  *   to determine eviction. If false, no check will be done and you may grow up memory
  *   usage. Disabled is default
+ * @param {boolean} storeClone If the data stored by `set()` should be cloned to prevent
+ *   future mutation from affecting cached data. Disabled is default
  */
 export function buildMemoryStorage(
   cloneData = false,
   cleanupInterval: number | false = false,
-  maxEntries: number | false = false
+  maxEntries: number | false = false,
+  storeClone = false
 ) {
   const storage = buildStorage({
     set: (key, value) => {
@@ -67,6 +70,16 @@ export function buildMemoryStorage(
         }
       }
 
+      if (storeClone && value !== undefined) {
+        // Clone the value before storing to prevent future mutations
+        // from affecting cached data.
+        if (typeof structuredClone === 'function') {
+          value = structuredClone(value);
+        } else {
+          value = JSON.parse(JSON.stringify(value)) as NotEmptyStorageValue;
+        }
+      }
+
       storage.data[key] = value;
     },
 

test/storage/memory.test.ts

@@ -38,6 +38,30 @@ describe('tests memory storage', () => {
     expect(result2.data?.data).toBe('data');
   });
 
+  // Expects that a when value saved using storage.set() is has his inner properties updated,
+  // a request to storage.get() should return unmodified value.
+  //
+  // https://github.com/arthurfiorette/axios-cache-interceptor/issues/580
+  it('not allow changes by value reference before', async () => {
+    const storage = buildMemoryStorage(false, false, false, true);
+
+    const data = { ...EMPTY_RESPONSE, data: 'data' };
+    await storage.set('key', {
+      state: 'cached',
+      createdAt: Date.now(),
+      ttl: 1000 * 60 * 5, // 5 Minutes
+      data: data
+    });
+
+    data.data = 'another data';
+
+    const result = (await storage.get('key')) as CachedStorageValue;
+
+    expect(result).not.toBeNull();
+    expect(result.state).toBe('cached');
+    expect(result.data.data).toBe('data');
+  });
+
   it('tests cleanup function', async () => {
     const storage = buildMemoryStorage(false, 500);