fix: Setting cookie: false is not respected (#1442)

* fix: Setting cookie: false is not respected

* fix: Setting cookie: false is not respected

Co-Authored-By: Brendan Mulholland <[email protected]>

* Update storage.ts

Co-authored-by: Brendan Mulholland <[email protected]>

Author: Intevel

src/core/storage.ts

@@ -232,6 +232,9 @@ export class Storage {
   // Cookies
   // ------------------------------------
   getCookies(): Record<string, unknown> {
+    if (!this.isCookiesEnabled()) {
+      return
+    }
     const cookieStr = process.client
       ? document.cookie
       : this.ctx.req.headers.cookie
@@ -248,6 +251,10 @@ export class Storage {
       return
     }
 
+    if (!this.isCookiesEnabled()) {
+      return
+    }
+
     const _prefix =
       options.prefix !== undefined ? options.prefix : this.options.cookie.prefix
     const _key = _prefix + key
@@ -293,6 +300,10 @@ export class Storage {
       return
     }
 
+    if (!this.isCookiesEnabled()) {
+      return
+    }
+
     const _key = this.options.cookie.prefix + key
 
     const cookies = this.getCookies()
@@ -345,4 +356,28 @@ export class Storage {
       return false
     }
   }
+
+  isCookiesEnabled(): boolean {
+    // Disabled by configuration
+    if (!this.options.cookie) {
+      return false
+    }
+
+    // Server can only assume cookies are enabled, it's up to the client browser
+    // to create them or not
+    if (process.server) {
+      return true
+    }
+
+    if (window.navigator.cookieEnabled) {
+      return true
+    } else {
+      // eslint-disable-next-line no-console
+      console.warn(
+        "[AUTH] Cookies is enabled in config, but browser doesn't" +
+          ' support it'
+      )
+      return false
+    }
+  }
 }