From ddab7b3fe9631f0d45efea4931b0fe40af13bbe1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Sat, 7 Mar 2026 07:59:40 +0800 Subject: [PATCH] Add missing path=/ to setCookieWithoutExpiration Co-Authored-By: Claude Opus 4.6 --- src/utils/cookieUtil.js | 2 +- tests/unit/utils/cookieUtil.test.js | 9 +++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/src/utils/cookieUtil.js b/src/utils/cookieUtil.js index 246ff2c..c172857 100644 --- a/src/utils/cookieUtil.js +++ b/src/utils/cookieUtil.js @@ -50,7 +50,7 @@ export function setCookie(name, value, days=1) { * @param {string} value - The cookie value. */ export function setCookieWithoutExpiration(name, value) { - document.cookie = name + "=" + (value || "") + "; Secure; SameSite=Lax"; + document.cookie = name + "=" + (value || "") + "; path=/; Secure; SameSite=Lax"; } /** diff --git a/tests/unit/utils/cookieUtil.test.js b/tests/unit/utils/cookieUtil.test.js index 57a47c9..5d98250 100644 --- a/tests/unit/utils/cookieUtil.test.js +++ b/tests/unit/utils/cookieUtil.test.js @@ -82,6 +82,15 @@ describe('cookieUtil', () => { expect(str).toContain('Secure'); expect(str).toContain('SameSite=Lax'); }); + + it('sets cookie with path=/', () => { + setCookieWithoutExpiration('pathKey', 'pathVal'); + + const written = cookieSetter.mock.calls.find( + (c) => c[0].startsWith('pathKey='), + ); + expect(written[0]).toContain('path=/'); + }); }); describe('deleteCookie', () => {