소셜 로그인 버그 FIX #169
Merged
소셜 로그인 버그 FIX #169
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sehee-Lee-01
approved these changes
Mar 14, 2024
There was a problem hiding this comment.
오늘 있엇던 이슈(#167)를 해결하는 코드도 있는 것 같군요!
전반적으로 메시지 상수화로 깔끔하게 해주시고 메서드 분리도 잘 해주셔서 잘 읽힌 것 같습니다!
어려운 기능 구현해주시느라 너무너무너무 수고 많으셨습니다....
Comment on lines
+39
to
+49
| private OAuth2UserImpl extractOAuth2User(Authentication authentication) { | ||
| if (!(authentication.getPrincipal() instanceof OAuth2UserImpl oauth2User)) { | ||
| throw new InternalAuthenticationServiceException(OAUTH_USER_TYPE_IS_INVALID); | ||
| } | ||
| return oauth2User; | ||
| } | ||
|
|
||
| private SocialLoginResponse createSocialLoginResponse(User user, AuthProvider authProvider) { | ||
| LoginResponse loginResponse = authService.createTokens(user); | ||
| return SocialLoginResponse.of(loginResponse, authProvider); | ||
| } |
📝 Jacoco Test Coverage
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎫 관련 이슈
Resolves #157
📌 기존 로그인 방식
1️⃣ 클라이언트(프론트엔드)에서
깃허브로 로그인하기버튼을 누르면, 백엔드 서버 uriGET /api/v1/oauth2/authorization/github로 요청!2️⃣ 백엔드 서버는 깃허브 로그인 페이지로 넘어갈 수 있는 리다이렉트 uri를 클라이언트에 응답! (oauth2-client 프레임워크가 지원해줌😎)
3️⃣ 클라이언트가 깃허브 로그인 성공!
4️⃣ 깃허브 서버가 지정된 리다이렉트 uri(백엔드 서버
/api/v1/login/oauth2/code/github?code={code}&state={state})로 code를 넘겨줌!5️⃣ 백엔드 서버가 code를 받아 처리한 뒤 로그인 응답을 내려줌! (oauth2-client 프레임워크가 지원해줌😎)
🚨 문제 상황
💬 해결책 2번 방식을 사용하면 code를 받아서 깃허브 서버에 access token을 요청하고, 기타 처리를 해주는 유용한 oauth2-client의 장점을 사용할 수 없어😥
💬 그럼 해결책 1은? 리다이렉트 응답을 보내면 body를 넣을 수 없어! 하지만 우린.. 로그인 성공 시 json 응답(토큰+유저 정보)를 내려줘야해!! body 대신 uri에 파라미터 형식으로 넣을수도 있고, 쿠키에 json을 넣을수도 있지만 보안상으로나 json 파싱이나 너무 복잡해..
💡 해결된 방식
사실 해결책 2번으로 해결이 되었다..😆 클라이언트가 깃허브 로그인 성공 시 코드를 받고, 백엔드 서버에 요청을 하는데, 기존 oauth2client에서 제공해주는 엔드포인트(
/login/oauth2/code/github?code={code}&state={state})를 그대로 쓸 수 있더라ㅎ대신
application.yml에 redirectUri를 클라이언트 uri로 추가해 주어야함!✅ 구현 내용
AuthExceptionHandlerFilter)OAuth2LoginFailureHandler)💬 코멘트