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

[✨feat] 소셜 로그인 관련 기능 구현 #83

Merged
merged 116 commits into from
Feb 26, 2025
Merged

[✨feat] 소셜 로그인 관련 기능 구현 #83

merged 116 commits into from
Feb 26, 2025
+2,867 −122

Conversation

0-tae
Copy link
Contributor

@0-tae 0-tae commented Feb 25, 2025

🚀 What’s this PR about?

  • 작업 내용 요약:
    • 회원가입 로직 수정
    • 소셜 로그인
    • 토큰 재발급
    • 인가코드를 통해 인증하기

🛠️ What’s been done?

  • 주요 변경사항을 상세히 기술하세요. (예: 새로운 기능 추가, 버그 수정, 코드 리팩토링 등)
    • 회원가입 코드 복원
    • 소셜 로그인 (Sign-In) API 구현
    • 클라이언트로 부터 인가 코드를 받고 인증하는 API 구현
    • AccessToken 재발급 구현
    • GlobalLogger 구현
    • AuthTokenResolver 구현 (신규 회원의 인증 정보 및 토큰 임시 저장)

🧪 Testing Details

  • 테스트 코드 및 결과: 작성한 테스트 코드와 주요 테스트 케이스를 설명하고, 통과된 테스트 결과를 요약해 주세요.
    • 기존 테스트 통과 하였습니다.
    • OauthService 관련 부분(회원가입, 토큰 재발급, 소셜 로그인, 인가 코드 인증) 은 통합 테스트로 대체했고, 테스트 통과 하였습니다.
      • 신규 사용자 일 경우
      • 기존 사용자인 경우
      • 액세스 토큰이 만료된 사용자의 경우
      • 리프레시 토큰이 만료된 사용자의 경우

👀 Checkpoints for Reviewers

  • 리뷰 시 확인할 사항:
  • 클라이언트에서 AuthType을 받지 않고 액세스 토큰을 O(1)로 인증 하는 방법이 없을까요?
    • OauthController의 reissue-token API처럼 액세스 토큰으로 소셜 사용자 인증을 수행할 때, AuthType이 제공되지 않아, 현재 가지고 있는 AuthType 모두를 순차적으로 조회하여 소셜 유저 정보 불러오기를 O(N)으로 수행합니다. 따라서 Kakao 토큰임에도 Google에 요청을 보내어 인증 실패를 겪은 후에 Kakao에 다시 요청을 보내어 인증 정보를 불러옵니다. 이로 인해 불필요한 외부 서버 연결이 일어나게 되는 문제가 발생하게 되는데, 자유롭게 의견 남겨주시면 감사 드리겠습니다.

🎯 Related Issues

0-tae added 30 commits February 17, 2025 17:56
@0-tae
fix(S3UploadError) : 이미지 업로드 IOException 예외처리
618993b 
- 관련 ErrorCode 메시지 추가
- try-catch 구문으로 예외 제어
@0-tae
fix(GroupCreate) : S3Service uploadImage 호출부 예외처리 제거
07d4c82
@0-tae
refactor(AuthInfo) : Member-AuthInfo 엔티티 분리
5ee0a11 
- AuthInfo가 Member를 외래키로 가지도록 설정
- Member에서 refreshToken, authId, authType 값을 분리
@0-tae
feat(AuthInfoRepository) : AuthInfoRepository 구현
a7032fa
@0-tae
feat(AuthException) : Auth 관련 예외처리 구현
7161726
@0-tae
feat(MemberService) : MemberService 기능 구현
d054d5c 
- 멤버 생성 createMember 메소드 구현
@0-tae
fix(GroupServiceTest) : Member-AuthInfo 분리로 인한 테스트 코드 수정
44ce8e5 
- Member 생성에 필요한 AuthInfo 관련 파라미터 삭제
@0-tae
feat(SignUpDto) : SignUp에 대한 Dto 클래스 구현
7ee7a18
@0-tae
chore(build.gradle) : JWT 관련 의존성 추가
9430193
@0-tae
feat(JwtTokenProvider) : JWT Token 및 Provider 구현
3034764
@0-tae
feat(JwtInterceptor) : JWT 인터셉터 구현
eb44e9e
@0-tae
feat(WebConfig) : WebConfig 설정
574c153 
- auth를 제외한 모든 경로에 적용
@0-tae
feat(AuthInfoService) : AuthInfoService 구현
8238944 
- AuthInfo 생성을 이한 createAuthInfo 구현
@0-tae
feat(KakaoService) : KakaoService 구현
ea8be1d 
- 카카오 서버 토큰 요청을 위한 requestToken 구현
@0-tae
feat(KakaoToken) : KakaoToken 관련 Dto 클래스 구현
465724e 
- 내부에 KakaoErrorInfo 구현하여 에러 여부 확인
@0-tae
feat(KakaoApi) : KakaoApi 관련 예외처리 및 상수 클래스 구현
cb6510e
@0-tae
refactor(AuthId) : AuthId -> Code로 명칭 변경
bc5acfd 
- 기존 Oauth 용어인 Code로 명칭 통일
@0-tae
refactor(ErrorResponse) : args 파라미터를 포함한 생성자 추가
db0a69a
@0-tae
refactor(Code) : 파라미터 및 멤버필드 code로 수정
3922f3a
@0-tae
refactor(AuthException, AuthErrorCode) : ErrorCode 상수 표시명 수정 및 BaseEx…
1c9ed75 
…ception 상속
@0-tae
feat(AuthId) : AuthId VO 클래스 구현
458d4cd
@0-tae
fix(AuthInfo) : AuthId VO 도입으로 인한 코드 수정
2e08cba
@0-tae
fix(AuthInfoRepository) : AuthId VO 도입으로 인한 코드 수정
a111cd0
@0-tae
test(AuthIdTest) : AuthId VO 도입으로 인한 코드 및 파일명 수정
768f272
@0-tae
refactor(AuthInfoService) : AuthId를 파라미터로 받는 메소드 추가
c369202
@0-tae
fix(AwsConfig) : S3Client awsSecret 및 액세스 키 설정
b9aeebc 
- S3Storage Bean 주입 메소드 작성
@0-tae
feat(S3ServiceImpl) : 의존성 주입을 위한 Qualifier 추가 및 Primary 컴포넌트 설정
8a7c648 
- 명칭 임시 지정(dev, prod)
@0-tae
refactor(BaseException) : BaseException 파라미터 주입을 위한 메소드 추가
8c18944
@0-tae
refactor(ErrorResponse) : ErrorResponse 생성자 수정
2f229b8 
- Object -> String 파라미터 타입 수정
@0-tae
fix(GlobalExceptionHandler) : log 기능 추가
bcd7a81
0-tae added 25 commits February 23, 2025 18:03
@0-tae
fix(OauthController) : 토큰 재발급(액세스 토큰) API 경로 수정
1007d88
@0-tae
Merge branch 'feat/69' into feat/56
6d02508 
# Conflicts:
#	src/main/java/org/noostak/auth/api/OauthController.java
@0-tae
feat(Authorize) : 인가 관련 DTO 클래스 구현
ffece48
@0-tae
feat(AuthSuccessCode) : 인가 관련 성공 응답 상수 선언
44d98ee
@0-tae
feat(RefreshToken) : refreshToken에 대해 null값이 될 수 없도록 변경
a3de66d 
- 관련하여 유효성 검증 수행
- 에러코드 추가
@0-tae
feat(AuthInfoService) : 기존 사용자 여부를 확인하고 Authorize 관련 응답을 반환하는 서비스 로직 구현
44a0458 
- 기존 사용자일 경우 refreshToken을 업데이트
@0-tae
refactor(GlobalLogger) : GlobalLogger 클래스 구현 및 Slf4j 어노테이션 제거
e294a1c
@0-tae
fix(JwtToken) : 미사용 import 구문 제거
68b348a
@0-tae
feat(SignUpRequest) : authId 필드 추가
02a8a9b
@0-tae
feat(AuthTokenResolver) : AuthTokenResolver 클래스 구현 및 빈 등록
a50c7c3 
- AuthId를 키 값으로 사용하여 JwtToken을 지정된 시간동안 유지하는 기능 구현
- 스케줄링을 통해 만료된 JwtToken과 AuthId 정보를 삭제하는 기능 구현
@0-tae
feat(AuthInfoService) : AuthTokenResolver 의존성 추가 및 서비스 로직 추가
e5ad6fd 
- 멤버가 아닐 경우 인증 정보를 임시적으로 유지하는 로직 추가
- authId를 이용해서 임시 정보를 불러오는 로직 추가
@0-tae
feat(AuthErrorCode) : 임시 정보 만료에 대한 에러 메시지 추가
8e4bee8
@0-tae
feat(OauthController) : 회원가입 로직 수정 및 OAuth 인증 API(authorize) 구현
d4cfede 
- 클라이언트에게서 authType과 인가 코드를 받아 소셜 서비스에 인증하여, 멤버 여부, authId, 토큰을 반환하는 API 구현
- 기존 signUp에서 인가 코드를 통해 소셜 서비스를 인증하는 로직 제거
- authId를 통해서 서버 메모리에 저장된jwtToken을 조회하는 흐름으로 변경
@0-tae
chore(CD) : oauth-config.yml 추가로 인한 워크플로우 수정
8478b65 
- oauth-config.yml 파일을 working-directory에 추가
@0-tae
Merge branch 'develop_merge' into feat/73
5e9a20b
@0-tae
fix(AuthId, AuthType) : 테스트 코드 컴파일 에러 수정
9e27791 
- AuthId, AuthType 도메인 변경으로 인한 패키지 import 수정
- findByMember_MemberId -> findByMemberId 수정
@0-tae
feat(AuthInfoServiceImpl) :findByAuthId 구현
c1f3816
@0-tae
feat(AuthTokenResolver) : 로깅 추가
53f90aa
@0-tae
refactor(JwtToken) : 생성자 파라미터 변경 및 정적 메소드 사용
5ada678
@0-tae
refactor(RefreshToken) : 유효성 검사 규칙 변경
9860a55 
- null을 허용할 수 없도록 변경
- 파라미터로 받은 token에 대하여 유효성 검증을 수행하도록 로직 변경
@0-tae
fix(AuthType) : 사용하지 않는 AuthType 삭제
9ae7ef4 
- APPLE 삭제
@0-tae
test(AuthInfoServiceImplTest) : 테스트 코드 작성 및 테스트
7195b34
@0-tae
test(FakeAuthInfoRepository,FakeAuthInfoService) : 모의 객체 테스트 코드 작성
3047f94
@0-tae
test(AuthTokenResolver) : 테스트 코드 작성 및 테스트
a527bff
@0-tae
fix(OauthController) : 로깅 메시지 수정
b0b5aaf
@0-tae 0-tae self-assigned this Feb 25, 2025
@0-tae 0-tae added ✨ feat Something isn't working 🦖 용태 🦖 Further information is requested labels Feb 25, 2025
@0-tae 0-tae changed the title [feat] 소셜 로그인 관련 기능 구현 [✨feat] 소셜 로그인 관련 기능 구현 Feb 25, 2025
@0-tae 0-tae merged commit e400500 into develop Feb 26, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@0-tae 0-tae

Labels
✨ feat Something isn't working 🦖 용태 🦖 Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@0-tae @jsoonworld