feat: Receive Kakao Server Access Token (OAuth2) #5

h-beeen · 2024-03-16T13:14:07Z

PULL REQUEST

🎋 작업 중인 브랜치

feat: Spring Security & Oauth2 구현 #1

🔑 주요 작업사항

Kakao Developer OAuth2 키 값 추가
OAuth 요청을 Provider Path를 활용해 클라이언트 별로 분기
현재, Kakao OAuth2 클라이언트의 providerId를 활용한 회원가입 및 JWT Access/Refresh 토큰 부여까지 구현

🏞 (Optional) 참고 자료

회원가입 플로우

클라이언트가 https://api.vacgom.co.kr/v1/oauth/kakao/ 엔드포인트 호출
서버에서 kakao Client에 맞는 Kakao 로그인 Uri로 리다이렉트 (Http Status 302)
로그인 Uri가 정상적으로 종료된다면, 해당 Kakao ProviderId 기반으로 회원가입 or 기존 가입 유저 찾기
해당 유저를 기준으로 JWT Access 토큰 발행 후 아래 응답 반환

{
  "member": {
    "memberId": "05d602f6-91f8-4ab0-8dc5-c6b6b71425ed", // UUID
    "role": "ROLE_TEMP_USER" // ROLE
  },
  "token": {
    "accessToken": "eyJhbGciOiJIUzI1NiJ9.eyJzdWIiOiIwNWQ2MDJmNi05MWY4LTRhYjAtOGRjNS1jNmI2YjcxNDI1ZWQiLCJpYXQiOjE3MTA2MzE4NTMsImV4cCI6MTcxMDY0MDg1M30.hNG0CG5zdk9NetVRRMQ******",
    "refreshToken": "eyJhbGciOiJIUzI1NiJ9.eyJzdWIiOiIwNWQ2MDJmNi05MWY4LTRhYjAtOGRjNS1jNmI2YjcxNDI1ZWQiLCJpYXQiOjE3MTA2MzE4NTMsImV4cCI6MTcxMTg0MTQ1M30.t96qYOQnVEJyBdGIWUy******"
  }
}

이후 클라이언트 요청에 따라, 메인 페이지 or 백신 조회 페이지로 리다이렉트 가능 (구현점 협의해야함)

꼭 확인해 주세요!!

access token Validation 로직 access token 로그인 추후 개발 예정 (refresh token을 활용한 토큰 재발급...굳이 필요할까요...?)
아직 java의 늪에 벗어나지 못해 이 아른거리는 코틀린 아가의 코드입니다.
코틀린스럽지 않은 문법에 대한 리뷰와 Suggestion 절찬환영!!
이번 코드는 시큐리티를 통째로 만드는 것이 목표이기 때문에 리뷰 양이 많을 것으로 생각됩니다.
다음 코드부터는 잘게 쪼개서 소규모 PR로 분기하겠습니다.

감사합니다 :)

HyungJu

너무 .. 너무 수고하셨습니다 ..
제가 감히 무슨 리뷰를 남겨도 되는지는 모르겠지만 ㅠㅠ

제가 봤을때 궁금했던 점이나, 개선했으면 좋겠는 점들을 간단히 남겨보았습니다!

감사합니다.. 🥹

+) 이거 작업 다 하시면 제가 형주식 .. 클린 아키텍처로 리팩토링을 해서 괜찮을지 의견을 한번 여쭤보고 싶은데 그래도 괜찮으실까요 ..?!

src/main/kotlin/com/vacgom/backend/application/auth/AuthService.kt

src/main/kotlin/com/vacgom/backend/domain/auth/oauth/constants/ProviderType.kt

src/main/kotlin/com/vacgom/backend/infrastructure/auth/oauth/kakao/KakaoConnector.kt

src/main/kotlin/com/vacgom/backend/infrastructure/auth/oauth/kakao/model/KakaoProperties.kt

HyungJu · 2024-03-17T04:13:00Z

src/main/kotlin/com/vacgom/backend/domain/auth/RefreshToken.kt

+    @Id
+    @GeneratedValue
+    @Column(name = "rt_id")
+    private val id: Long? = null


id도 다른 프라퍼티들 처럼 생성자 자리에서 정의하셔도 문제 없습니다 !

그리고 RefreshToken 엔티티는 BaseEntity 상속을 안받고 있는데 의도된 것인지도 궁금합니다 !!

@HyungJu 궁금한 점이 있습니다!!
생성자를 통해 RefreshToken을 생성하려고 할 때,
id와 같은 GeneratedValue Property를 null로 할당해 작업하나요?
객체 생성 파라미터에 null을 넣는게 어색해서 분리해보았는데, 요 부분도 의견 부탁드릴게요!!

그리고 RefreshToken 엔티티는 BaseEntity 상속을 안받고 있는데 의도된 것인지도 궁금합니다 !!

RefreshToken역시 BaseEntity 상속을 받아야 맞습니다!
요 부분도 반영해둘게요!!
p.s.Redis 마이그레이션 염두중..

@HyungJu 궁금한 점이 있습니다!! 생성자를 통해 RefreshToken을 생성하려고 할 때, id와 같은 GeneratedValue Property를 null로 할당해 작업하나요? 객체 생성 파라미터에 null을 넣는게 어색해서 분리해보았는데, 요 부분도 의견 부탁드릴게요!!

사실 전 Domain Model -> Entity 간 변환을 할때 id도 함께 매핑을 해줘야 하기 때문에 .. 그렇게 하고 있어요

(이미지 참조)

아니면 named argument를 사용하면 조금 보기 좋게 표현할수도 있습니다 ! (named argument를 사용하면 default value가 지정되어 있을때 해당 argument는 생략할 수 있어요)

오호 이 부분은 처음 접하는 정보인데요..
어떤 느낌인지는 알 것 같습니다!
요 부분도 #8 PR에서 반영해볼게요!!

src/main/kotlin/com/vacgom/backend/global/security/jwt/JwtProvider.kt

src/main/kotlin/com/vacgom/backend/infrastructure/auth/oauth/kakao/KakaoConnector.kt

HyungJu · 2024-03-17T04:46:52Z

제가 리뷰 하는 방법도 더 배워와서 다음 리뷰는 좀 더 잘 .. 해 보겠습니다 .. 항상 감사합니다 ..

HyungJu · 2024-03-17T04:48:06Z

(refresh token을 활용한 토큰 재발급...굳이 필요할까요...?)
-> 저도 굉장히 동의하는 것이 .. ! 지금 단계에서는 길이가 적당히 긴 엑세스 토큰만 발급해도 무리는 없어보입니다 .. !

src/main/kotlin/com/vacgom/backend/global/exception/ApiExceptionHandler.kt

* feat: JWT 연동 완료 및 구조 개선 * feat: OAuth2 구현 완료 * feat: 코틀린스러운 문법 수정

h-beeen added 4 commits March 15, 2024 15:00

feat: Member Entity 필요 속성 추가

b9f894b

feat: Configuration 계층 수정

13b2d35

feat: Init Kakao Oauth / Redirect API

70c687d

feat: OAuth2 Provider 기준 분기 후 로그인 API로 Redirect

91fa771

h-beeen added the ✨ Feat 새로운 기능을 추가하는 경우 label Mar 16, 2024

h-beeen added this to the [Global] Security Logic milestone Mar 16, 2024

h-beeen requested a review from HyungJu March 16, 2024 13:14

h-beeen self-assigned this Mar 16, 2024

feat: JWT 기본 로직 / ProviderId 기반 회원가입/로그인 로직

266069d

h-beeen force-pushed the feat/#1 branch from ff482a9 to 266069d Compare March 16, 2024 23:14

h-beeen added 2 commits March 17, 2024 08:22

chore: 패키지 구조 개선

b88fc88

feat: JWT 연동 완료 및 구조 개선

a1b2c68

Base automatically changed from develop to master March 17, 2024 02:38

HyungJu requested changes Mar 17, 2024

View reviewed changes

h-beeen linked an issue Mar 17, 2024 that may be closed by this pull request

feat: Spring Security & Oauth2 구현 #1

Closed

h-beeen commented Mar 17, 2024

View reviewed changes

src/main/kotlin/com/vacgom/backend/global/exception/ApiExceptionHandler.kt Outdated Show resolved Hide resolved

h-beeen mentioned this pull request Mar 17, 2024

feat: OAtuh / Security Logic 코드리뷰 반영 #8

Merged

h-beeen added 2 commits March 18, 2024 20:12

feat: OAtuh / Security Logic 코드리뷰 반영 (#8)

dd3e100

* feat: JWT 연동 완료 및 구조 개선 * feat: OAuth2 구현 완료 * feat: 코틀린스러운 문법 수정

Merge remote-tracking branch 'origin/develop' into feat/#1

674e84b

h-beeen force-pushed the feat/#1 branch from 8c53a76 to 674e84b Compare March 18, 2024 11:21

h-beeen changed the base branch from master to develop March 18, 2024 11:22

h-beeen merged commit abd5688 into develop Mar 18, 2024
1 check passed

h-beeen deleted the feat/#1 branch March 18, 2024 11:24

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: Receive Kakao Server Access Token (OAuth2) #5

feat: Receive Kakao Server Access Token (OAuth2) #5

h-beeen commented Mar 16, 2024 •

edited

Loading

HyungJu left a comment

HyungJu Mar 17, 2024

HyungJu Mar 17, 2024

h-beeen Mar 17, 2024 •

edited

Loading

h-beeen Mar 17, 2024 •

edited

Loading

HyungJu Mar 17, 2024 •

edited

Loading

h-beeen Mar 17, 2024

HyungJu commented Mar 17, 2024

HyungJu commented Mar 17, 2024 •

edited

Loading

feat: Receive Kakao Server Access Token (OAuth2) #5

feat: Receive Kakao Server Access Token (OAuth2) #5

Conversation

h-beeen commented Mar 16, 2024 • edited Loading

PULL REQUEST

🎋 작업 중인 브랜치

🔑 주요 작업사항

🏞 (Optional) 참고 자료

관련 이슈

꼭 확인해 주세요!!

HyungJu left a comment

Choose a reason for hiding this comment

HyungJu Mar 17, 2024

Choose a reason for hiding this comment

HyungJu Mar 17, 2024

Choose a reason for hiding this comment

h-beeen Mar 17, 2024 • edited Loading

Choose a reason for hiding this comment

h-beeen Mar 17, 2024 • edited Loading

Choose a reason for hiding this comment

HyungJu Mar 17, 2024 • edited Loading

Choose a reason for hiding this comment

h-beeen Mar 17, 2024

Choose a reason for hiding this comment

HyungJu commented Mar 17, 2024

HyungJu commented Mar 17, 2024 • edited Loading

h-beeen commented Mar 16, 2024 •

edited

Loading

h-beeen Mar 17, 2024 •

edited

Loading

h-beeen Mar 17, 2024 •

edited

Loading

HyungJu Mar 17, 2024 •

edited

Loading

HyungJu commented Mar 17, 2024 •

edited

Loading