로그인 로직 개발 #53

sjhjack · 2024-12-29T12:49:33Z

📑 개요

로그인 관련 비즈니스 로직 작성한다. #41

✅ PR 체크리스트

🔀 PR 제목의 형식을 잘 작성했나요?
💯 테스트는 잘 통과했나요?
🏗️ 빌드는 성공했나요?
🧹 불필요한 코드는 제거했나요?
💭 이슈는 등록했나요?
🏷️ 라벨은 등록했나요?

🚀 상세 작업 내용

UserInfo 요청 Google API 추가
Birthday 요청 Google API 추가
- 생일 비공개 유저 저장 로직 추가
GoogleOAuthClient 모듈의 public 메서드에 설명 주석 추가
AuthController, AuthService 작성
로그인/회원가입 기능 구현
- 로그인/회원가입 분기 처리
  - DB 조회 O : 로그인 처리 (토큰 정보 저장)
  - DB 조회 X : 회원가입 처리 (회원 정보, 토큰 정보 저장)
토큰 재발급 기능 구현
- 토큰 유효성 검증 분기 처리
  - Redis 조회 O : 토큰 재발급 에러 (401)
  - Redis 조회 X : 잘못된 토큰 에러 (400)
회원 탈퇴 기능 구현
- Redis에서 토큰 정보 삭제
- 회원 정보 Hard Delete -> 추후 논의 후 변경 예정
Redis Token 모듈 작성
- {key : value} = {access_token : refresh_token} 으로 저장 (refresh_token 만료기한 만큼)
- 토큰 정보 저장, 삭제, 조회 기능 작성
전체 테스트 코드 작성

📁 ETC

closed: #41

…드 추가

KIMSEI1124

해당 이슈 처리하시느라 고생하셨습니다! 👍🏼

몇 가지가지 궁금한 사항들작성해 두었고었수정해야 하는하는 목록에 대해서도 코멘트를 달아두었습니다!
하시다가 궁금하신 내용이 있다질문해 주시면시면 감사합니다!

application/wypl-core/src/main/java/com/wypl/wyplcore/auth/controller/AuthController.java

application/wypl-core/src/main/java/com/wypl/wyplcore/auth/service/AuthServiceImpl.java

application/wypl-core/src/test/java/com/wypl/wyplcore/auth/controller/AuthControllerTest.java

domain/redis-token-domain/src/test/java/com/wypl/redistokendomain/TokenRepositoryTest.java

KIMSEI1124 · 2024-12-31T04:29:39Z

domain/jpa-member-domain/src/test/java/com/wypl/jpamemberdomain/SocialMemberRepositoryTest.java

+			assertThat(result.getId()).isEqualTo(findSocialMember.getId());
+			assertThat(result.getMember().getMemberId()).isEqualTo(findSocialMember.getMember().getMemberId());
+			assertThat(result.getOauthId()).isEqualTo(findSocialMember.getOauthId());
+			assertThat(result.getOauthProvider()).isEqualTo(findSocialMember.getOauthProvider());


이렇게 하나씩 하는 것 보다 재귀적으로 확인할 수 있는 테스트 방법이 있습니다!

해당 방법으로 수정해보시고, 발생하는 문제점을 해결해보시면 좋을 것 같습니다!

확인해보니 isEqualTo() 메서드는 Object끼리 비교가 가능하여 assertThat(result).isEqualTo(findSocialMember) 으로 수정했습니다!
의도하신 방법이 맞을까요?

다른 방법을 의도하였는데! 해당 방법으로 성공이 되었군요! 아마도 BaseEntity를 extends 하지 않아서 그런 것 같습니다!

assertThat(foundMember) .usingRecursiveComparison() .ignoringFields("created", "lastModified") .isEqualTo(savedMember);

다음과 같이 객체 내부의 필드 값들에 대해서 재귀적으로 검증하면서 일부 검증이 불필요한 내용을 제외하는 방법이 있습니다! 추후 다른 테스트를 진행하실 때 사용해 보면 좋을 것 같아요!

이런 방법도 있었군요! 배워갑니다. 감사합니다 😄

KIMSEI1124 · 2024-12-31T04:30:22Z

...-domain/src/main/java/com/wypl/jpamemberdomain/member/utils/SocialMemberRepositoryUtils.java

+	public static SocialMember getSocialMember(SocialMemberRepository socialMemberRepository, String id) {
+		return socialMemberRepository.findByOauthProviderAndOauthId(OauthProvider.GOOGLE, id)
+			.orElseThrow(() -> new MemberException(MemberErrorCode.NO_SUCH_MEMBER));
+	}


해당 내용을 보아하니 구글에 종속이 되어 보입니다!

만약 다른 로그인 방식이 생긴다면 어떻게 처리하실건가요?!

좋은 지적 감사합니다!
현재 생각으로는 OauthProvider를 파라미터로 전달받도록 할 것 같습니다.
메서드를 사용하는 모든 곳에서 수정해야 한다는 번거로움이 있지만, 아직 다른 방법이 떠오르지 않습니다 😢

혹시 고려해볼 방법이 있을까요?

사실 파라미터를 제외하면 생각할 방법이 메서드의 이름에다 어떤 제공자에게 인증을 받았는지 적어주는 방법이 생각납니다!

public static SocialMember getSocialMemberByGoogle(SocialMemberRepository socialMemberRepository, String id) { return socialMemberRepository.findByOauthProviderAndOauthId(OauthProvider.GOOGLE, id) .orElseThrow(() -> new MemberException(MemberErrorCode.NO_SUCH_MEMBER)); }

하지만 새로운 제공자가 생기면 메서드가 하나씩 늘어난다는 단점이 있습니다! 한번 고민해 보시고 선택하면 좋을 것 같아요!

domain/jpa-member-domain/src/main/java/com/wypl/jpamemberdomain/member/data/MemberDto.java

...in/jpa-member-domain/src/main/java/com/wypl/jpamemberdomain/member/data/SocialMemberDto.java

KIMSEI1124 · 2024-12-31T04:32:43Z

domain/auth-domain/src/test/java/com/wypl/authdomain/auth/service/fixture/MemberFixture.java

+// Todo : Access Level 고민해보자
+// @AllArgsConstructor


주석 내용을 해결해주시면 감사합니다!

@AllArgsConstructor가 인식되지 않는 원인을 아직 해결하지 못해서 해당 부분 주석 처리 후 생성자를 명시적으로 추가해놓은 상태입니다.
해결하는 대로 더 고민해보겠습니다!

domain/auth-domain/src/main/java/com/wypl/authdomain/auth/service/AuthDomainServiceImpl.java

clean test -> test -i로 변경

isEqualTo(false) -> isFalse()

.github/workflows/backend-server-test.yml

github-actions · 2025-01-03T14:05:00Z

📄 Jacoco Coverage Report

Overall Project	92.77%	🍏

There is no coverage information present for the Files changed

KIMSEI1124

전체적으로 잘 된 것 같습니다!!

해당 이슈 개발하느라 엄청 고생많으셨습니다! 👍🏼
먼저 Approve를 해두었습니다! 몇몇 희망사항을 적어봤는데 한번 봐주시면 좋을 것 같습니다!

KIMSEI1124 · 2025-01-05T08:11:27Z

application/wypl-core/src/main/java/com/wypl/wyplcore/auth/service/AuthServiceImpl.java

+
+	@Transactional
+	public void quitMember(AuthMember authMember) {
+		// Todo : 회원 탈퇴 로직 논의


해당 논의가 필요하다는 내용으로 이슈를 하나 만들어주세요! 👍🏼

정리해서 올려보겠습니다!

KIMSEI1124 · 2025-01-05T08:12:42Z

application/wypl-core/src/test/java/com/wypl/wyplcore/auth/controller/AuthControllerTest.java

+@AutoConfigureRestDocs
+// @AutoConfigureMockMvc
+// @SpringBootTest
+@WebMvcTest(AuthController.class)


Good! 따로 작성하신 문서를 봤는데 잘 해결하신 것 같아요!

그렇다면 @ControllerTest 와 같은 어노테이션을 만들어서 처리할 수 있을 것 같은데 따로 이슈를 만들어서 해결해보는건 어떨까요?!

좋은 의견입니다!
두 어노테이션을 합해서 만들어보겠습니다.

application/wypl-core/src/test/resources/application.yml

Dto의 의도가 명확하지 않다는 피드백을 반영하여 아래와 같이 변경했습니다. - MemberDto -> MemberSaveDto - SocialMemberDto -> SocialMemberSaveDto

실행 속도는 느려지지만 최신 상태에서 테스트 하는 것이 더 중요하다고 생각하여 clean 명령어 추가

github-actions · 2025-01-05T13:42:23Z

📄 Jacoco Coverage Report

Overall Project	92.77%	🍏

There is no coverage information present for the Files changed

sjhjack added 30 commits November 9, 2024 17:01

Move(#41): AuthMemberServiceImpl 위치 변경

b13b50d

Fix(#41): SocialMember 쿼리 생성 오류 해결

cba69c6

Refactor(#41): AuthMemberServiceImplTest의 SocialMember 생성 코드 수정

872cdda

Move(#41): SocialMemberRepository 위치 변경

2bc8a5f

Feat(#41): Google UserInfo 요청 API 추가

f17b362

Refactor(#41): SocialMemberRepository import 위치 변경

69b8cf7

Feat(#41): SocialMemberRepositoryUtils 추가

3ca284d

Refactor(#41): utils의 static 메서드 적용

bd5d5d7

Feat(#41): AuthTokensResponse 추가

2cf8bf5

Feat(#41): GoogleUserInfoResponse 추가

3ad11ab

Refactor(#41): Google Token 요청 주석 작성

76cc592

Refactor(#41): Google Token 재발급 요청 주석 작성

31d9b90

Refactor(#41): Google 유저 정보 요청 주석 작성

eb5b014

Refactor(#41): Google 토큰 유효성 검증 요청 주석 작성

e13e40a

Feat(#41): auth-domain, redis-token-domain 모듈 추가

1ce9a0c

Feat(#41): AuthService 토큰 재발급 메서드 작성

abd6b44

Feat(#41): Redis 설정 추가

778c5ce

Feat(#41): auth-domain 예외 처리 추가

cd0f5bf

Refactor(#41): 생성되는 RedisTemplate 이름 변경

dada50d

Feat(#41): wypl-core에 auth-domain 의존성 추가

0c1b0c4

Feat(#41): AuthServiceImpl 에 로그인/회원가입 로직 추가

955844a

Feat(#41): SocialMemberRepository 에 exits 쿼리 추가

63fd27f

Feat(#41): AuthDomainServiceImpl 에 토큰 존재 여부 확인, 회원가입 데이터 저장, 토큰 저장 메서…

5361fdc

…드 추가

Feat(#41): 토큰 저장 로직 추가

9f58ed7

Refactor(#41): memberId 로직 모듈화

6e66277

Feat(#41): 로그인 로직에 토큰 저장 로직 추가

dd21ad0

Refactor(#41): not exists -> exists로 변경

7d55c16

Feat(#41): 생일 요청 API의 response 추가

7b37dee

Feat(#41): GoogleOAuthClient에 생일 요청 API 추가

3712e72

Fix(#41): #48 충돌 해결

7829a5c

sjhjack linked an issue Dec 29, 2024 that may be closed by this pull request

로그인 관련 비즈니스 로직 작성한다. #41

Closed

8 tasks

KIMSEI1124 requested changes Dec 31, 2024

View reviewed changes

sjhjack requested a review from KIMSEI1124 January 1, 2025 09:33

sjhjack and others added 6 commits January 1, 2025 19:16

Update backend-server-test.yml

addce75

clean test -> test -i로 변경

Chore(#41): 사용하지 않는 어노테이션 삭제

245eb97

Chore(#41): 재발행 관련 고민내용 주석 삭제

33de446

Refactor(#41): false 검증 메서드 변경

a1c9768

isEqualTo(false) -> isFalse()

Refactor(#41): 누락된 @transactional 추가 및 readOnly 속성 추가

8c7b0ed

Refactor(#41): SocialMember 동등비교 로직 수정

344a3ea

KIMSEI1124 reviewed Jan 1, 2025

View reviewed changes

.github/workflows/backend-server-test.yml Outdated Show resolved Hide resolved

sjhjack added 6 commits January 3, 2025 21:22

Chore(#41): application.yml에 test 프로필 추가

4574152

Chore(#41): main의 application.yml에서 test 프로필 삭제

030e256

Feat(#41): test 용 yml 파일 추가

02658ce

Refactor(#41): AuthControllerTest에 WebMvcTest 적용

0d4de83

Chore(#41): test 용 application.yml 주석 처리

7b57b3f

Feat(#41): test 용 application 추가

dc3af27

KIMSEI1124 approved these changes Jan 5, 2025

View reviewed changes

sjhjack added 6 commits January 5, 2025 21:33

Remove(#41): 깃허브 에러 해결 후 필요 없는 파일 제거

527dc9d

Rename(#41): Dto 클래스 이름 명확히 변경

548a0d0

Dto의 의도가 명확하지 않다는 피드백을 반영하여 아래와 같이 변경했습니다. - MemberDto -> MemberSaveDto - SocialMemberDto -> SocialMemberSaveDto

Refactor(#41): git workflows 테스트 명령어에 clean 추가

1c3563f

실행 속도는 느려지지만 최신 상태에서 테스트 하는 것이 더 중요하다고 생각하여 clean 명령어 추가

Chore(#41): 주석 제거

736bc06

Refactor(#41): @ContextConfiguration 추가

237043b

Chore(#41): test 용 더미 파일 추가

b32aad7

sjhjack merged commit b176f05 into dev Jan 7, 2025
4 checks passed

sjhjack deleted the feat/#41 branch January 7, 2025 10:50

This was referenced Jan 9, 2025

@ControllerTest 어노테이션 생성한다. #54

Open

회원 서비스 관련 기능을 분리한다. #57

Open

sjhjack mentioned this pull request Jan 16, 2025

[BUG] 깃허브 액션 테스트 단계에서 yml 파일을 인식하지 못하는 오류를 해결한다. #61

Open

2 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

로그인 로직 개발 #53

로그인 로직 개발 #53

sjhjack commented Dec 29, 2024

KIMSEI1124 left a comment •

edited

Loading

KIMSEI1124 Dec 31, 2024

sjhjack Jan 1, 2025

KIMSEI1124 Jan 1, 2025 •

edited

Loading

sjhjack Jan 5, 2025

KIMSEI1124 Dec 31, 2024

sjhjack Jan 1, 2025

KIMSEI1124 Jan 1, 2025

KIMSEI1124 Dec 31, 2024

sjhjack Jan 1, 2025

github-actions bot commented Jan 3, 2025

KIMSEI1124 left a comment

KIMSEI1124 Jan 5, 2025

sjhjack Jan 5, 2025

KIMSEI1124 Jan 5, 2025

sjhjack Jan 5, 2025

github-actions bot commented Jan 5, 2025

		// Todo : Access Level 고민해보자
		// @AllArgsConstructor

로그인 로직 개발 #53

로그인 로직 개발 #53

Conversation

sjhjack commented Dec 29, 2024

📑 개요

✅ PR 체크리스트

🚀 상세 작업 내용

📁 ETC

KIMSEI1124 left a comment • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

KIMSEI1124 Jan 1, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Jan 3, 2025

📄 Jacoco Coverage Report

KIMSEI1124 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

github-actions bot commented Jan 5, 2025

📄 Jacoco Coverage Report

KIMSEI1124 left a comment •

edited

Loading

KIMSEI1124 Jan 1, 2025 •

edited

Loading