[feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 #13

dudxo · 2024-08-01T13:16:12Z

📑 작업 상세 내용

시큐리티 세팅
- SecurityConfig는 지속적으로 수정될 예정입니다.
소셜 로그인 API 구현
- 로그인 성공 시 신규/기존 회원 구분을 통해 서로 다른 url로 이동하도록 했습니다.
  - 신규/기존 회원 구분을 위한 Auth Entity 추가했습니다.
  - ERD도 반영했습니다.
- 회원가입 requestDto의 Validation은 Validation 라이브러리 미추가로 인해 진행하지 않았습니다.(Gradle 충돌 방지)
- Redis 미연동으로 Refresh 관리 부분 및 토큰 재발급 부분이 아직 미구현 상태입니다.
- security Oauth2 사용과 JWT 키 생성을 위해 application.yml 수정했습니다.
  - 노션 application.yml에도 수정했습니다.
- API 테스트를 위한 인증 객체 SetUp 메서드 작성

💫 작업 요약

SecurityConfig 작성
카카오 로그인 구현
JWT 구현(80%)
회원가입 로직 구현(공무원 이메일 인증 X)
API 테스트를 위한 인증 객체 SetUp 메서드 작성

🔍 중점적으로 리뷰 할 부분

AccessToken과 RefreshToken의 만료 시간에 대해 확인 부탁드립니다.
CustomOauth2SuccessHandler.class에서 request.setHeader()를 통해 인증 토큰을 담은 뒤 리다이랙션을 시키고 있는데, 이를 프론트엔드 측에서 확인할 수 있는지 잘 모르겠습니다.
CustomOauth2SuccessHandler.class isAuthStatusOld()를 굳이 extractMethod 한건지 확인 부탁드립니다.
CustomOauth2UserService.class 리팩토링 진행하며 SRP를 지켜보았는데 괜찮은지 확인 부탁드립니다.

- Provider로부터 획득한 유저 정보 처리 Service 구현

- 인증 객체 Custom Dto 추가 - AuthDto.class, CustomOauth2User.class

- 네이버 소셜로그인 도입을 고려한 Oauth2Response interface 구현 - 카카오 소셜로그인을 위한 Oauth2Response 구현체 KakaoResponse 구현

- socialEmail로 회원을 찾는 쿼리 메서드 구현

- Custom Authentication 객체 구현

- 신규 유저 또는 기존 유저 구분 후 저장 또는 업데이트 로직 메서드 추출 - 단일 책임 원칙(SRP)를 위한 책임 분배(createSocialName(), createMemberFromOauth2Response()) - MemberRepository.findBySocialEmail() return Type 변경(Member -> Optional<Member>)

- Member로 Auth 찾는 쿼리 메서드 추가

- Auth 저장 또는 업데이트 메서드 구현 - 기존 회원 구분 메서드 구현

- 회원 구분하여 신규 회원일시 추가 정보 기입 페이지로 리다이렉트 구현

- saveOrUpdate(Oauth2Response oauth2Response) : CustomeOauth2userService.class -> MemberService.class

- KAKKOA -> KAKAO 수정

- kakao login api 구현

hyun2371

수정하느라 고생 많으셨습니다!!

src/main/java/com/dnd/gongmuin/auth/domain/AuthStatus.java

src/main/java/com/dnd/gongmuin/auth/service/AuthService.java

src/main/java/com/dnd/gongmuin/member/controller/MemberController.java

src/main/java/com/dnd/gongmuin/member/exception/MemberErrorCode.java

src/main/java/com/dnd/gongmuin/member/service/MemberService.java

dudxo · 2024-08-02T16:55:40Z

@hyun2371 수정사항 현재 다 수정했습니다!

낮에 물어보신 @AuthenticationPrincipal 를 통해 인증객체를 꺼낼 때 Member로 바로 꺼낼 수 있게 해주시면 안되냐고 하셨는데,
인증 객체 생성을 CustomOauth2User -> Member로 변경함에 따라 가능하도록 변경헀습니다!
JWT 인증을 통해 제대로 확인 해야되는데 현재 JWT부분이 100% 미완성이라서 추후 확인이 필요해보입니다!

hyun2371 · 2024-08-02T17:02:39Z

@hyun2371 수정사항 현재 다 수정했습니다!

낮에 물어보신 @AuthenticationPrincipal 를 통해 인증객체를 꺼낼 때 Member로 바로 꺼낼 수 있게 해주시면 안되냐고 하셨는데,
인증 객체 생성을 CustomOauth2User -> Member로 변경함에 따라 가능하도록 변경헀습니다!
JWT 인증을 통해 제대로 확인 해야되는데 현재 JWT부분이 100% 미완성이라서 추후 확인이 필요해보입니다!

오 감사합니다🙂 늦은 시간까지 고생많으셨어요!!👍

src/test/java/com/dnd/gongmuin/security/jwt/TokenProviderTest.java

src/main/java/com/dnd/gongmuin/auth/domain/Auth.java

src/test/java/com/dnd/gongmuin/auth/service/AuthServiceTest.java

src/test/java/com/dnd/gongmuin/member/repository/MemberRepositoryTest.java

- Auth - Member 로딩 전략 변경(Eager -> Lazy) - 가독성을 위한 static import 제거

src/main/java/com/dnd/gongmuin/member/service/MemberService.java

src/test/java/com/dnd/gongmuin/security/jwt/TokenProviderTest.java

src/main/java/com/dnd/gongmuin/member/service/MemberService.java

dudxo added 30 commits July 25, 2024 17:28

[feat] : Security Config 작성

23591d7

[feat] : member Repository 추가

1a86bf4

[feat] : Oauth2UserSerivce 구현

be602ff

- Provider로부터 획득한 유저 정보 처리 Service 구현

[feat] : AuthErrorCode 구현

4c542bb

[feat] : 인증 객체 Dto 추가

4a5dab1

- 인증 객체 Custom Dto 추가 - AuthDto.class, CustomOauth2User.class

[feat] : Oauth2UserInfo Mapping 객체 구현

87ab12a

- 네이버 소셜로그인 도입을 고려한 Oauth2Response interface 구현 - 카카오 소셜로그인을 위한 Oauth2Response 구현체 KakaoResponse 구현

[style] : AuthDto 네이밍 변경으로 인한 수정

8fbffb3

[feat] : socialEmail update method 추가

df1d40e

[feat] : Member Repository 추가

3c65791

- socialEmail로 회원을 찾는 쿼리 메서드 구현

[test] : member socialEmail udpate method 테스트 구현

4351bf7

[test] : memberRepository.findBySocialEmail() 테스트 구현

e5e4127

[feat] : CustomOauth2User 구현

00bbd45

- Custom Authentication 객체 구현

[feat] MemberService.class 추가

a14d0f7

[test] : 쿼리메서드 타입 변경으로 인한 테스트 코드 수정

41665d5

[feat] : 회원 구분 객체 Auth 엔티티 추가

2cdd4b7

[feat] : AuthRepository 추가

e706c88

- Member로 Auth 찾는 쿼리 메서드 추가

[feat] AuthService 추가

d3b7fe7

- Auth 저장 또는 업데이트 메서드 구현 - 기존 회원 구분 메서드 구현

[feat] : AuthErrorCode 에러코드 추가

6435102

[feat] : Auth 객체 저장 또는 업데이트 로직 추가

1859885

[feat] : SocialName 필드에서 Provider 추출 메서드 구현

d6189ed

[feat] : OauthLogin 성공 핸들러 추가

60de8a7

- 회원 구분하여 신규 회원일시 추가 정보 기입 페이지로 리다이렉트 구현

[refactor] : 단일 책임 원칙 준수하기 위한 메서드 이동

e6e2b26

- saveOrUpdate(Oauth2Response oauth2Response) : CustomeOauth2userService.class -> MemberService.class

[fix] : Auth 생성 시 연관된 회원 주입

e723028

[fix] : 오탈자로 인한 오류 해결

e314026

- KAKKOA -> KAKAO 수정

[feat] : MemberErrorCode 추가

55d4697

[feat] : AuthController 추가

f658a13

- kakao login api 구현

[fix] : 회원 검색 실패 시 예외 추가

c6c62e9

[fix] : provider명 변경

e7eb2d6

[fix] : Kakao Login Api 명세와 일치되도록 수정

19a3123

[style] : import 클래스명 변경

95a5875

hyun2371 reviewed Aug 2, 2024

View reviewed changes

dudxo added 10 commits August 3, 2024 00:13

[feat] : requestDto validation 추가

90a4ee3

[rename] : enum 필드명 통일을 위한 네이밍 수정

90745c3

[refactor] : 가독성을 위한 static import enum타입 제거

9006df8

[test] : 가독성을 위한 static import enum타입 제거

9bac4dc

[rename] : enum 에러코드 네이밍 수정

1c70ad0

[feat] : requestMapping 추가

288f1a1

[feat] : 추가정보 api 인증/인가 허용

2d1b09b

[sytle] : 안쓰는 의존 코드 삭제

7e50d2e

[feat] : 소셜이메일로 회원 찾는 메서드 추가

4806bcb

[fix] : CustomOauth2User 객체가 아닌 Member 객체를 인증 객채로 생성되도록 변경

e99cf34

hyun2371 reviewed Aug 3, 2024

View reviewed changes

src/test/java/com/dnd/gongmuin/security/jwt/TokenProviderTest.java Outdated Show resolved Hide resolved

dudxo added 7 commits August 3, 2024 17:02

[test] : CI test 오류로 인한 테스트 disable 처리

8b58c67

[test] : 주입 클래스, 변수 순서 변경

adce9d9

[test] : mock 테스트 명시를 위한 어노테이션 수정

e9808eb

[refactor] : AuthInfo of() Builder를 이용한 객체 생성 변경

2fb8677

[feat] : 모든 필드 값을 주입하여 생성하는 팩터리 메서드 추가

f12de11

[test] : Member 관련 test를 위한 MemberFixture 추가

3f3ae1f

[test] : API 테스트를 위한 인증 객체 setUp 추가

128ce98

dudxo requested a review from hyun2371 August 3, 2024 16:36

hyun2371 approved these changes Aug 3, 2024

View reviewed changes

hyun2371 changed the title ~~[feat] : 시큐리티 세팅 및 소셜 로그인 API 구현~~ [feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 Aug 4, 2024

dudxo added 2 commits August 4, 2024 12:54

[feat] Auth 도메인 로딩 전략 변경

250b9ca

- Auth - Member 로딩 전략 변경(Eager -> Lazy) - 가독성을 위한 static import 제거

[test] : 통합 테스트용 추상메서드 상속

b565df1

dudxo merged commit 5b1615f into dev Aug 4, 2024
3 checks passed

dudxo deleted the feat/#4/social-login branch August 5, 2024 11:48

hyun2371 reviewed Aug 11, 2024

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 #13

[feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 #13

dudxo commented Aug 1, 2024 •

edited

Loading

hyun2371 left a comment

dudxo commented Aug 2, 2024

hyun2371 commented Aug 2, 2024

[feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 #13

[feat #4] : 시큐리티 세팅 및 소셜 로그인 API 구현 #13

Conversation

dudxo commented Aug 1, 2024 • edited Loading

관련 이슈

📑 작업 상세 내용

💫 작업 요약

🔍 중점적으로 리뷰 할 부분

hyun2371 left a comment

Choose a reason for hiding this comment

dudxo commented Aug 2, 2024

hyun2371 commented Aug 2, 2024

dudxo commented Aug 1, 2024 •

edited

Loading