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.

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

Arch 22 feat/social login #25

Merged
merged 45 commits into from
Jan 12, 2024
Merged

Conversation

GaBaljaintheroom
Copy link
Collaborator

작업 내용 (Content)

  • 카카오 소셜 로그인 기능 추가
  • 구글 소셜 로그인 기능 추가
  • Member 엔티티 수정
  • flyway 문서 추가

링크 (Links)

기타 사항 (Etc)

  • 구글 로그인 scope는 profile, image이고 kakao 로그인은 profile_image, account_email, 닉네임도 있어서 Member 엔티티에 닉네임을 어떻게 저장해야 할지에 대해 -> 일단 구글과 카카오의 scope를 이메일과, 프로필 이미지로 맞추고 닉네임은 전화번호 인증 후 업데이트를 하는 방식으로 생각함
  • 소셜 로그인 맡은 부분을 알맞게 구현을 한건지 의문

Merge 전 필요 작업 (Checklist before merge)

  • config application-local.yml 수정

희망 리뷰 완료 일 (Expected due date)

202X. X. X. Wed

@GaBaljaintheroom GaBaljaintheroom self-assigned this Jan 10, 2024
@GaBaljaintheroom GaBaljaintheroom added the feat 새로운 기능 label Jan 10, 2024
@seokho-1116
Copy link
Collaborator

OAuth2 url을 주는 엔드포인트도 만들어야 할듯?

@seokho-1116
Copy link
Collaborator

OAuth2용 Spring filter chain을 하나 붙여야 할듯

@seokho-1116
Copy link
Collaborator

OAuth2AuthorizationRequestRedirectFilter 관련해서 리서치

Copy link
Collaborator

@seokho-1116 seokho-1116 left a comment

Choose a reason for hiding this comment

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

굿

@seokho-1116 seokho-1116 merged commit 6baca9f into develop_back Jan 12, 2024
2 checks passed
@seokho-1116 seokho-1116 deleted the ARCH-22-feat/social_login branch January 25, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat 새로운 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants