Skip to content

fix(provider): omit Kakao email scope when optional#2579

Open
pjpjq wants to merge 1 commit into
supabase:masterfrom
pjpjq:codex/kakao-email-scope
Open

fix(provider): omit Kakao email scope when optional#2579
pjpjq wants to merge 1 commit into
supabase:masterfrom
pjpjq:codex/kakao-email-scope

Conversation

@pjpjq

@pjpjq pjpjq commented Jun 16, 2026

Copy link
Copy Markdown

What changed

  • Keep Kakao's existing default behavior of requesting account_email.
  • Omit account_email from the default Kakao scopes when the provider is configured with email_optional.
  • Preserve an explicit opt-in path by allowing custom scopes to add account_email back.
  • Add provider-level scope tests and external Kakao callback tests for the no-email flow.

Why

Supabase's Kakao docs explain that developers can omit account_email and enable "Allow users without an email". The provider currently always requests account_email, which blocks non-Biz Kakao apps before Supabase can handle the no-email user flow.

Fixes #2574.

Validation

  • go test ./internal/api/provider
  • go test ./internal/api -run 'TestExternal/TestSignupExternalKakao'
  • git diff --check

@pjpjq pjpjq requested a review from a team as a code owner June 16, 2026 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Kakao provider hardcodes account_email scope, blocking non-Biz Korean developers

1 participant