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/#11] 구글 로그인 API 구현 #13

Merged
merged 36 commits into from
Jan 20, 2025
Merged

[FEAT/#11] 구글 로그인 API 구현 #13

merged 36 commits into from
Jan 20, 2025

Conversation

gahyuun
Copy link
Member

@gahyuun gahyuun commented Jan 14, 2025 •
edited
Loading

💡 Issue

📄 Description

구글 로그인

  • 사용자의 id token을 받으면 해당 id token을 가지고 검증을 한 이후에 사용자 정보를 가져오는 로직입니다
  • 이때 idToken이 null 일땐 사용자가 보낸 id token의 검증이 실패한 것이므로 400 error 반환 그 외는 내부 시스템 문제라고 판단 후 500 에러 반환

ConfigurationProperties

  • 여러 개 값을 바인딩하는 경우 유용할 것 같아 @Value 보다 @ConfigurationProperties를 사용했습니다

💬 To Reviewers

  • @Transactional을 중간에 오류났을 때 롤백하기 위해 login 함수에만 사용했습니다!
  • 혹시 다른 함수에도 필요한지, 잘못 사용했는지 피드백 주실 부분 있으면 편하게 주세요 ~!

MemberController에서 MemberService 의존성 주입이 안되는 문제가 있는데 ( private과 final이 왜 없죠 .. ) 이 부분은 다른 브랜치에서 수정했으니 무시해주세요~

🔗 Reference

@gahyuun gahyuun self-assigned this Jan 14, 2025
@gahyuun gahyuun added the size/M label Jan 14, 2025
@gahyuun gahyuun changed the title Feat/#11 [Feat/#11] 구글 로그인 구현 Jan 15, 2025
@gahyuun gahyuun added the ✨ FEAT 새로운 기능 추가 label Jan 15, 2025
@gahyuun gahyuun marked this pull request as ready for review January 15, 2025 05:14
@gahyuun gahyuun requested a review from ckkim817 as a code owner January 15, 2025 05:14
@gahyuun gahyuun changed the title [Feat/#11] 구글 로그인 구현 [Feat/#11] 구글 로그인 API 구현 Jan 15, 2025
@gahyuun gahyuun changed the title [Feat/#11] 구글 로그인 API 구현 [FEAT/#11] 구글 로그인 API 구현 Jan 15, 2025
@gahyuun gahyuun added size/L and removed size/M labels Jan 16, 2025
@ckkim817 ckkim817 merged commit 4c5d5ab into develop Jan 20, 2025
1 check passed
@ckkim817 ckkim817 deleted the feat/#11 branch January 20, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ckkim817 ckkim817 Awaiting requested review from ckkim817 ckkim817 is a code owner

Assignees

@gahyuun gahyuun

Labels
✨ FEAT 새로운 기능 추가 size/L 🦊 가현
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FEAT] 구글 로그인 API 구현
2 participants
@gahyuun @ckkim817