Comparing cff37d400d..86f254e939 - web

groombook/web

Fork 0

Author	SHA1	Message	Date
Stockboy Steve	86f254e939	fix(GRO-2572): follow Better Auth redirect URL from signIn.social response CI / Test (pull_request) Successful in 20s Details CI / Lint & Typecheck (pull_request) Successful in 25s Details CI / Build & Push Docker Image (pull_request) Successful in 47s Details Better Auth's signIn.social() returns { data: { redirect: true, url } } rather than issuing an HTTP 30x when using the fetch client. The LoginPage handler was discarding data.url, so the SSO button appeared to do nothing (the button disabled but the user never left /login). Fix: after the social sign-in call, if result.data.url is present, navigate via window.location.href. Also add an early return in the error branch so the two paths don't bleed into each other. Updated UAT_PLAYBOOK.md §5.4.1 TC-WEB-SSO-2 to require a fresh/incognito context so a stale auth cookie can't mask the regression. Co-Authored-By: Paperclip <noreply@paperclip.ing>	2026-06-26 12:05:38 +00:00

Author

SHA1

Message

Date

Stockboy Steve

86f254e939

fix(GRO-2572): follow Better Auth redirect URL from signIn.social response

CI / Test (pull_request) Successful in 20s

Details

CI / Lint & Typecheck (pull_request) Successful in 25s

Details

CI / Build & Push Docker Image (pull_request) Successful in 47s

Details

Better Auth's signIn.social() returns { data: { redirect: true, url } } rather
than issuing an HTTP 30x when using the fetch client. The LoginPage handler
was discarding data.url, so the SSO button appeared to do nothing (the button
disabled but the user never left /login).

Fix: after the social sign-in call, if result.data.url is present, navigate via
window.location.href. Also add an early return in the error branch so the two
paths don't bleed into each other.

Updated UAT_PLAYBOOK.md §5.4.1 TC-WEB-SSO-2 to require a fresh/incognito
context so a stale auth cookie can't mask the regression.

Co-Authored-By: Paperclip <noreply@paperclip.ing>

2026-06-26 12:05:38 +00:00

Compare commits

1 Commits

dev .. fix/gro-2572-sso-redirect

Diff Content Not Available

Compare commits

1 Commits dev .. fix/gro-2572-sso-redirect

Diff Content Not Available

1 Commits

dev .. fix/gro-2572-sso-redirect