Merge pull request #3 from streamsqldb/fix/cookie-path

fix(cookies): set default cookie path to /

Author: ecmarsh

.size-snapshot.json

@@ -1,26 +1,26 @@
 {
   "dist/streamsql.esm.js": {
-    "bundled": 8662,
-    "minified": 4273,
-    "gzipped": 1705,
+    "bundled": 8725,
+    "minified": 4301,
+    "gzipped": 1721,
     "treeshaked": {
       "rollup": {
         "code": 75,
         "import_statements": 0
       },
       "webpack": {
-        "code": 4866
+        "code": 4894
       }
     }
   },
   "dist/streamsql.cjs.js": {
-    "bundled": 10213,
-    "minified": 4843,
-    "gzipped": 1782
+    "bundled": 10276,
+    "minified": 4871,
+    "gzipped": 1798
   },
   "dist/streamsql.min.js": {
-    "bundled": 11154,
-    "minified": 4647,
-    "gzipped": 1748
+    "bundled": 11227,
+    "minified": 4675,
+    "gzipped": 1765
   }
 }

integration/fixtures/streamsql.min.js

@@ -47,6 +47,7 @@ export default class CookieJar implements CookieHandler {
       oneYearFromNow.setFullYear(oneYearFromNow.getFullYear() + 1)
       cookieOpts.expires = oneYearFromNow.toUTCString()
     }
+    cookieOpts.path = cookieOpts.path || '/'
     cookieOpts.secure = true
     cookieOpts.sameSite = SameSite.None
     this.simpleCookies.set(cookieOpts.name, cookieOpts.value)
@@ -85,6 +86,7 @@ export default class CookieJar implements CookieHandler {
     this.setCookie({
       name,
       value: '',
+      path: '/',
       maxAge: 0,
       expires: epoch.toUTCString()
     })

src/Cookies.ts

@@ -76,6 +76,14 @@ describe('CookieHandler', () => {
     expect(document.cookie).toMatch('Secure')
   })
 
+  it('sets default cookie path to /', () => {
+    ch.setCookie({
+      name: 'name',
+      value: 'value',
+    })
+    expect(document.cookie).toMatch('Path=/')
+  })
+
   it('overrides default domain and path when set in options', () => {
     ch.setCookie({
       name: 'name',