Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Refactor: 일반 로그인 사용자와 소셜 로그인 사용자의 구분을 위한 리팩토링 #236

Merged
merged 1 commit into from
Aug 6, 2024
Merged

[Refactor: 일반 로그인 사용자와 소셜 로그인 사용자의 구분을 위한 리팩토링 #236

merged 1 commit into from
Aug 6, 2024

Conversation

runtime-zer0
Copy link
Contributor

@runtime-zer0 runtime-zer0 commented Aug 6, 2024 •
edited
Loading

#️⃣연관된 이슈

이슈번호: #234

📝작업 내용

일반 로그인 사용자와 소셜 로그인 사용자의 구분을 위한 리팩토링

  • MemberType.java

    • 로그인 사용자의 타입 구분하여 관리하기 위한 Enumerate 클래스 생성
    • 열거형 상수 필드: REGULAR("regular"), SOCIAL("social")
    • 필드 value

  • Member.java

    • memberType 필드 추가
    • 생성자 및 buildMember() 메서드 수정

  • MemberResponse.ProfileDTO.java

    • memberType 필드 추가

  • MemberRepositoryCustomImpl.java

    • findMemberDTOByUsername() 메서드 쿼리 수정

  • MemberService.java

    • registerMember(), createAdminAccount() 메서드 내에 memberType 필드 추가로 인한 로직 수정

  • CustomOAuth2UserService.java

    • loadUser() 메서드 내에 소셜 로그인 사용자 생성시 파라미터 추가

  • CustomAuthenticationSuccessHandler.java

    • onAuthenticationSuccess() 매서드 내에 로그인에 성공시 클라이언트로 전달하기 위한 사용자의 정보 추가

  • AuthService.java

    • isAuthenticated() 메서드 내에 클라이언트가 사용자의 로그인 상태를 확인시 전달하는 사용자의 정보 추가

💬리뷰 요구사항

  • 일반 로그인 사용자와 소셜 로그인 사용자의 구분을 위한 리팩토링입니다.

@runtime-zer0
Refactor: 일반 로그인 사용자와 소셜 로그인 사용자의 구분을 위한 리팩토링
0db649e 
- MemberType.java
- 로그인 사용자의 타입 구분하여 관리하기 위한 Enumerate 클래스 생성
- 열거형 상수 필드: REGULAR("regular"), SOCIAL("social")
- 필드 value

- Member.java
- memberType 필드 추가
- 생성자 및 buildMember() 메서드 수정

- MemberResponse.ProfileDTO.java
- memberType 필드 추가

- MemberRepositoryCustomImpl.java
- findMemberDTOByUsername() 메서드 쿼리 수정

- MemberService.java
- registerMember(), createAdminAccount() 메서드 내에 memberType 필드 추가로 인한 로직 수정

- CustomOAuth2UserService.java
- loadUser() 메서드 내에 소셜 로그인 사용자 생성시 파라미터 추가

- CustomAuthenticationSuccessHandler.java
- onAuthenticationSuccess() 매서드 내에 로그인에 성공시 클라이언트로 전달하기 위한 사용자의 정보 추가

- AuthService.java
- isAuthenticated() 메서드 내에 클라이언트가 사용자의 로그인 상태를 확인시 전달하는 사용자의 정보 추가
@runtime-zer0 runtime-zer0 added the refactoring 코드 리팩토링 label Aug 6, 2024
@runtime-zer0 runtime-zer0 added this to the 4차 기능 구현 milestone Aug 6, 2024
@runtime-zer0 runtime-zer0 self-assigned this Aug 6, 2024
@runtime-zer0 runtime-zer0 merged commit 7c5f14f into main Aug 6, 2024
@runtime-zer0 runtime-zer0 linked an issue Aug 6, 2024 that may be closed by this pull request
[Refactor] 로그인 및 소셜 로그인 사용자 구분 관련 로직 리팩토링 #234
Open
3 tasks
@runtime-zer0 runtime-zer0 requested review from geunhokinn, meanzi3 and serahissomi and removed request for geunhokinn and meanzi3 August 6, 2024 10:15
geunhokinn
geunhokinn reviewed Aug 6, 2024
View reviewed changes
Copy link
Contributor

@geunhokinn geunhokinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다. 리팩토링 하시느라 고생하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@geunhokinn geunhokinn geunhokinn left review comments

@serahissomi serahissomi Awaiting requested review from serahissomi

Assignees

@runtime-zer0 runtime-zer0

Labels
refactoring 코드 리팩토링
Projects
None yet
Milestone
4차 기능 구현
Development

Successfully merging this pull request may close these issues.

[Refactor] 로그인 및 소셜 로그인 사용자 구분 관련 로직 리팩토링
2 participants
@runtime-zer0 @geunhokinn