opencollective-frontend icon indicating copy to clipboard operation
opencollective-frontend copied to clipboard

Use session cookies and logged in user in ssr

Open hdiniz opened this issue 1 year ago • 1 comments

Related https://github.com/opencollective/opencollective/issues/7423 Require https://github.com/opencollective/opencollective-api/pull/10165

hdiniz avatar Jun 26 '24 12:06 hdiniz

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
opencollective-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 11:35am
opencollective-styleguide ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 12, 2024 11:35am

vercel[bot] avatar Jun 26 '24 12:06 vercel[bot]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
opencollective-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2024 0:39am
opencollective-styleguide ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 26, 2024 0:39am

Ntriiss13 avatar Jul 06 '24 22:07 Ntriiss13

Looks good to me! Not sure if we need to update the places that are calling the confirmEmail and setPassword mutations? It worked out of the box when I tested an email change, I assume passing the new token to getLoggedInUser will end up exchanging it on the API side and getting a cookie back

Yes, the API is using the token in the Authorization header before the cookie.

hdiniz avatar Jul 30 '24 11:07 hdiniz