Skip to content

[Feat/jwt login]#27

Merged
ekgns33 merged 7 commits into
mainfrom
feat/jwt-login
Apr 7, 2025
Merged

[Feat/jwt login]#27
ekgns33 merged 7 commits into
mainfrom
feat/jwt-login

Conversation

@ekgns33
Copy link
Copy Markdown
Contributor

@ekgns33 ekgns33 commented Apr 7, 2025

작업 내역

1. 인증 기능 유저도메인에서 분리

  • 기존에 user에 존재하던 auth를 auth 패키지로 모두 이전했습니다.
    user에서는 이제 단순히 userRegisterService로 저장하는 것만 담당합니다.

2.소셜로그인 리팩토링

  • 소셜 제공자마다 oidc 형식이 다른 이슈가 있습니다. 추상화 했던 로직을 다시 구체화하여
    카카오와 애플로그인을 분리했습니다. endpoint 또한 나뉘게 됩니다.

  • /api/v1/auth/kakao

  • /api/v1/auth/apple


3.인증과정에서 예외처리 개선

AS

  • 예외처리가 중구난방으로 저마다 다른 Exception을 던지는 상황이었습니다.

TO BE

  • verifier, handler에서 Exception을 catch하여 CustomException을 throw하도록 변경했습니다.

4. Token 리프레쉬 로직 구현

  • 액세스토큰에 대해 리프레쉬 토큰으로 재 발급하는 로직을 구현했습니다.

PR이 너무 크네요... 옮기는 작업이랑 구현을 동시에 해서 너무 커졌습니다. 죄송합니다 ㅠ

@ekgns33 ekgns33 requested a review from jeeheaG April 7, 2025 05:55
@ekgns33 ekgns33 self-assigned this Apr 7, 2025
@ekgns33 ekgns33 added bug Something isn't working enhancement New feature or request 🚀 feature 🚀 and removed bug Something isn't working labels Apr 7, 2025
@ekgns33 ekgns33 merged commit 685f505 into main Apr 7, 2025
3 checks passed
@ekgns33 ekgns33 deleted the feat/jwt-login branch April 16, 2025 07:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request 🚀 feature 🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant