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

[feat] oauth2를 이용한 로그인 구현 #10

Merged
merged 19 commits into from
Oct 15, 2024

Conversation

ehs208
Copy link
Member

@ehs208 ehs208 commented Oct 13, 2024

1. 무슨 이유로 코드를 변경했나요?

OAuth2 및 이메일 인증 관련 로직을 구현 했습니다.

2. 어떤 위험이나 장애를 발견했나요?

OAuth2 관련 로직을 미완성 하였습니다.

3. 관련 스크린샷을 첨부해주세요.

image image

4. 완료 사항

이메일 인증 구현

5. 추가 사항

OAuth2 관련된 부분은, 일부 구현하였으나 프론트엔드와 협업을 통해 추후 배포서버에서 작동하도록 재구현 할 예정입니다.

OAuth를 통한 로그인 초기구현 완료
추가 정보 입력을 위한 로직 구현이 필요합니다.
추후 구현 예정
추후 고도화 예정입니다
key, value값 수정
예외처리를 포함하여 구현 완료
ouath 및 mail관련 내용 추가
이메일 및 패스워드 노출을 제거 후 수정했습니다
@ehs208 ehs208 linked an issue Oct 13, 2024 that may be closed by this pull request
@ehs208 ehs208 self-assigned this Oct 13, 2024
@ehs208 ehs208 added the Feat 새로운 기능을 추가합니다 label Oct 13, 2024
@ehs208 ehs208 requested review from yechan-kim and dyk-im October 13, 2024 15:50
Copy link
Member

@yechan-kim yechan-kim left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 코드를 읽어보고 의견 남겨두었으니 확인 부탁드립니다!

Copy link
Collaborator

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

저번 자체 로그인부터 어스 로그인 까지 구현 하시느라 고생하셨습니다!! 코드 확인하고 궁금한 부분만 따로 리뷰 남기겠습니다!!

@dyk-im dyk-im self-requested a review October 15, 2024 06:07
기존의 try-catch문을 없애고, GlobalHandlerException이 처리하게 변경하였습니다.
안정성을 위해 null을 return 하지 않는 방식으로 변경하였습니다.
Copy link
Member

@yechan-kim yechan-kim left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@ehs208 ehs208 merged commit 26b1746 into develop Oct 15, 2024
@ehs208 ehs208 deleted the feat/#9/oauth2를-이용한-로그인-구현 branch October 15, 2024 14:28
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.

[feat] OAuth2를 이용한 로그인 구현
3 participants