fix(local-storage): set http headers correctly

projects/ngneat/cashew/src/lib/local-storage/local-storage-cache.ts

@@ -1,4 +1,4 @@
-import { HttpResponse } from '@angular/common/http';
+import { HttpResponse, HttpHeaders } from '@angular/common/http';
 import { Injectable } from '@angular/core';
 import { HttpCacheStorage } from '../cache-storage';
 import { storage } from './local-storage';
@@ -12,35 +12,51 @@ function createKey(key: string) {
 @Injectable()
 export class HttpCacheLocalStorage extends HttpCacheStorage {
   has(key: string): boolean {
-    return super.has(createKey(key)) || !!storage.getItem(createKey(key));
+    const generatedKey = createKey(key);
+
+    return super.has(generatedKey) || !!storage.getItem(generatedKey);
   }
 
   get(key: string): HttpResponse<any> | boolean {
-    const cacheValue = super.get(createKey(key));
+    const generatedKey = createKey(key);
+
+    const cacheValue = super.get(generatedKey);
 
     if (cacheValue) {
       return cacheValue;
     }
 
-    const value = storage.getItem(createKey(key));
+    const value = storage.getItem(generatedKey);
 
     if (value) {
-      super.set(createKey(key), new HttpResponse(value));
+      value.headers = new HttpHeaders(value.headers);
+      value.headers.init();
+      const response = new HttpResponse(value);
+      super.set(generatedKey, response);
     }
 
-    return super.get(createKey(key))!;
+    return super.get(generatedKey)!;
   }
 
   set(key: string, response: HttpResponse<any>) {
-    storage.setItem(createKey(key), response);
+    const generatedKey = createKey(key);
+
+    const httpResponse: any = { ...response, headers: {} };
+    response.headers.keys().forEach(headerKey => {
+      httpResponse.headers[headerKey] = response.headers.get(headerKey);
+    });
+
+    storage.setItem(generatedKey, httpResponse);
 
-    return super.set(createKey(key), response);
+    return super.set(generatedKey, response);
   }
 
   delete(key: string) {
-    storage.clearItem(createKey(key));
+    const generatedKey = createKey(key);
+
+    storage.clearItem(generatedKey);
 
-    return super.delete(createKey(key));
+    return super.delete(generatedKey);
   }
 
   clear() {

projects/ngneat/cashew/src/lib/specs/mocks.ts

@@ -1,4 +1,4 @@
-import { HttpRequest, HttpResponse } from '@angular/common/http';
+import { HttpHeaders, HttpRequest, HttpResponse } from '@angular/common/http';
 import { CacheBucket } from '../cache-bucket';
 import { defaultConfig } from '../cache-config';
 import { DefaultHttpCacheGuard } from '../cache-guard';
@@ -16,7 +16,10 @@ export const ttl = config.ttl;
 
 export const httpRequest = (method: string = 'GET', options: any = {}, url: string = 'api/mock') =>
   new HttpRequest(method, url, {}, options);
-export const httpResponse = () => new HttpResponse();
+export const httpResponse = () =>
+  new HttpResponse({
+    headers: new HttpHeaders().set('test', 'test')
+  });
 export const requestQueue = () => new RequestsQueue();
 export const cacheBucket = () => new CacheBucket();
 export const httpCacheStorage = () => new DefaultHttpCacheStorage();