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

fix: oauth 로그인시 refreshToken 정보 저장 #226

Merged
merged 2 commits into from
Jan 11, 2024
Merged

fix: oauth 로그인시 refreshToken 정보 저장 #226

merged 2 commits into from
Jan 11, 2024

Conversation

eunbc
Copy link
Collaborator

@eunbc eunbc commented Jan 11, 2024

구현 기능

  1. oauth 로그인 성공 -> accessToken, refreshToken 발급
  2. refreshToken 을 redis에 저장

버그

/api/v1/auth/refresh로 토큰 재발급 하려고 하면

{
  "errorCode": "NOT ALLOWED BY PROVIDER",
  "errorMessage": "소셜 로그인 회원은 불가능한 요청입니다."
}

에러 발생하는데 확인 부탁드려요~!

소셜 로그인 회원은 비밀번호가 null인 로그인 요청을 보낼 수 없게 validation 처리 해놓은거 같은데,
토큰 재발급도 안됩니다!

resolve: #185

@eunbc eunbc added fix 수정 member 회원 labels Jan 11, 2024
@eunbc eunbc requested a review from kimday0326 January 11, 2024 10:47
@eunbc eunbc self-assigned this Jan 11, 2024
@kimday0326
Copy link
Member

kimday0326 commented Jan 11, 2024

구현 기능

  1. oauth 로그인 성공 -> accessToken, refreshToken 발급
  2. refreshToken 을 redis에 저장

버그

/api/v1/auth/refresh로 토큰 재발급 하려고 하면

{
  "errorCode": "NOT ALLOWED BY PROVIDER",
  "errorMessage": "소셜 로그인 회원은 불가능한 요청입니다."
}

에러 발생하는데 확인 부탁드려요~!

소셜 로그인 회원은 비밀번호가 null인 로그인 요청을 보낼 수 없게 validation 처리 해놓은거 같은데, 토큰 재발급도 안됩니다!

resolve: #185

리프레시 토큰 발급이 가능하도록 일부 수정하였는데 테스트 후 정상동작하면 머지 부탁드리겠습니다!

변경내용

CustomAuthenticationProvider -> MemberUserDetailsService에서 수행되던 소셜로그인 회원 검사 로직을
CustomAuthenticationProvider 로 빼내 분리하였습니다.

그 과정에서 UserDetailsImpl 필드가 추가되었고, 기존 코드와의 충돌을 방지하고자 생성자를 오버로딩하였습니다.

Copy link
Member

@kimday0326 kimday0326 left a comment

Choose a reason for hiding this comment

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

정상 동작 확인하고 머지 부탁드리겠습니다!🙇‍♀️

@eunbc eunbc merged commit 19e5bfb into main Jan 11, 2024
1 check passed
@eunbc eunbc deleted the member/185 branch January 11, 2024 12:39
@eunbc
Copy link
Collaborator Author

eunbc commented Jan 11, 2024

감사합니다 정상 동작 확인했습니다~!
머지 완료 💯

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 수정 member 회원
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

fix: oauth 로그인시 redis에 refreshToken 저장
2 participants