Merge pull request #1031 from jay-hodgson/set-cookie-domain

SWC-6826: must set to sub domain to share cookie
Sage-Bionetworks · Jun 26, 2024 · 4c47e68 · 4c47e68
2 parents 6260d0a + ab9f721
commit 4c47e68
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/apps/SageAccountWeb/src/components/AccountSettings.tsx b/apps/SageAccountWeb/src/components/AccountSettings.tsx
@@ -101,9 +101,11 @@ export const AccountSettings = () => {
     const current = new Date()
     const nextYear = new Date()
     nextYear.setFullYear(current.getFullYear() + 1)
+    const hostname = window.location.hostname.toLowerCase()
     cookies.set(SynapseConstants.DATETIME_UTC_COOKIE_KEY, isUTCTime, {
       path: '/',
       expires: nextYear,
+      domain: hostname.endsWith('.synapse.org') ? 'synapse.org' : undefined,
     })
   }, [isUTCTime])
 

diff --git a/packages/synapse-react-client/src/utils/hooks/useCookiePreferences.ts b/packages/synapse-react-client/src/utils/hooks/useCookiePreferences.ts
@@ -58,9 +58,11 @@ export const useCookiePreferences = (): [
       const current = new Date()
       const nextYear = new Date()
       nextYear.setFullYear(current.getFullYear() + 1)
+      const hostname = window.location.hostname.toLowerCase()
       cookies.set(COOKIES_AGREEMENT_COOKIE_KEY, prefs, {
         path: '/',
         expires: nextYear,
+        domain: hostname.endsWith('.synapse.org') ? 'synapse.org' : undefined,
       })
 
       setCookiePreferencesAtomValue(prefs)