Skip to content

Conversation

esinx
Copy link
Contributor

@esinx esinx commented Jul 31, 2025

What kind of change does this PR introduce?

This PR adds Kakao(https://naver.com/) as an external provider.

What is the current behavior?

This provider did not exist before.

What is the new behavior?

Based on Naver developer docs(https://developers.naver.com/docs/login/api/api.md), this PR creates a provider & test suite for Naver external provider.

Additional context

This was mentioned before #1251, but the PR was closed due to inactivity.
Someone asked for this during supabase launch week 15 @ seoul 😄

@esinx esinx requested a review from a team as a code owner July 31, 2025 12:37
@esinx esinx changed the title Naver Auth Provider feat: Naver Auth Provider Jul 31, 2025
@coveralls
Copy link

Pull Request Test Coverage Report for Build 16663254924

Details

  • 59 of 65 (90.77%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.08%) to 70.627%

Changes Missing Coverage Covered Lines Changed/Added Lines %
internal/api/provider/naver.go 56 62 90.32%
Totals Coverage Status
Change from base Build 16618040645: 0.08%
Covered Lines: 11912
Relevant Lines: 16866

💛 - Coveralls

@hf
Copy link
Contributor

hf commented Aug 4, 2025

We don't have anyone speaking Korean in the team, making fixes to this in the future hard. We'd like to support this via generic OAuth provider support which is currently being planned and high on the roadmap.

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.

3 participants