Fix cookie.set option defaults

nationalarchives · Sep 18, 2023 · 2db49a3 · 2db49a3
1 parent 22d8ce0
commit 2db49a3
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/nationalarchives/lib/cookies.mjs b/src/nationalarchives/lib/cookies.mjs
@@ -47,13 +47,15 @@ export default class Cookies {
     return decodeURIComponent(this.all[key]);
   }
 
-  set(key, value, options) {
-    const { maxAge, path, sameSite } = options;
+  set(key, value, options = {}) {
+    const {
+      maxAge = 60 * 60 * 24 * 365,
+      path = "/",
+      sameSite = "Lax",
+    } = options;
     document.cookie = `${encodeURIComponent(key)}=${encodeURIComponent(
       value,
-    )}; SameSite=${sameSite || "Lax"}; path=${path || "/"}; max-age=${
-      maxAge || 60 * 60 * 24 * 365
-    }; Secure`;
+    )}; SameSite=${sameSite}; path=${path}; max-age=${maxAge}; Secure`;
     this.#getPolicies();
   }
 
@@ -62,7 +64,7 @@ export default class Cookies {
   }
 
   get allPolicies() {
-    return JSON.parse(this.get(this.cookiesPolicyKey));
+    return JSON.parse(this.get(this.cookiesPolicyKey) || "{}");
   }
 
   policy(policy) {