-
Notifications
You must be signed in to change notification settings - Fork 1
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
The head ref may contain hidden characters: "feat/#9/oauth2\uB97C-\uC774\uC6A9\uD55C-\uB85C\uADF8\uC778-\uAD6C\uD604"
Conversation
OAuth를 통한 로그인 초기구현 완료 추가 정보 입력을 위한 로직 구현이 필요합니다.
예외처리를 포함하여 구현 완료
ouath 및 mail관련 내용 추가
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다! 코드를 읽어보고 의견 남겨두었으니 확인 부탁드립니다!
src/main/java/com/leets/xcellentbe/domain/user/domain/User.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/email/EmailService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/email/EmailService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/email/EmailService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/email/EmailService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/email/RedisService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/login/AuthController.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/login/oauth/CustomOAuthUserService.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/login/oauth/OAuthAttributes.java
Outdated
Show resolved
Hide resolved
src/main/java/com/leets/xcellentbe/global/auth/login/oauth/OAuthLoginSuccessHandler.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
저번 자체 로그인부터 어스 로그인 까지 구현 하시느라 고생하셨습니다!! 코드 확인하고 궁금한 부분만 따로 리뷰 남기겠습니다!!
src/main/java/com/leets/xcellentbe/global/auth/email/AuthCodeAlreadySentException.java
Outdated
Show resolved
Hide resolved
기존의 try-catch문을 없애고, GlobalHandlerException이 처리하게 변경하였습니다.
안정성을 위해 null을 return 하지 않는 방식으로 변경하였습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다!
1. 무슨 이유로 코드를 변경했나요?
OAuth2 및 이메일 인증 관련 로직을 구현 했습니다.
2. 어떤 위험이나 장애를 발견했나요?
OAuth2 관련 로직을 미완성 하였습니다.
3. 관련 스크린샷을 첨부해주세요.
4. 완료 사항
이메일 인증 구현
5. 추가 사항
OAuth2 관련된 부분은, 일부 구현하였으나 프론트엔드와 협업을 통해 추후 배포서버에서 작동하도록 재구현 할 예정입니다.