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: sign-in api를 추가한다. #16

Merged
merged 4 commits into from
Aug 4, 2024
Merged

feat: sign-in api를 추가한다. #16

merged 4 commits into from
Aug 4, 2024

Conversation

rlarltj
Copy link
Member

@rlarltj rlarltj commented Aug 3, 2024

📄 변경 사항

  • sign-in api를 추가합니다.
  1. OAuth Server로부터 User Data를 받아옵니다.
  2. Provider, OAuthID로 User DB를 조회합니다 (getOrRegister)
  3. AT, RT를 발급합니다.

@rlarltj rlarltj requested a review from min429 August 3, 2024 15:20
@rlarltj rlarltj self-assigned this Aug 3, 2024
@rlarltj rlarltj merged commit dc618ca into main Aug 4, 2024
1 check passed
@rlarltj rlarltj deleted the ISSUE-95 branch August 4, 2024 08:41
@min429
Copy link
Collaborator

min429 commented Aug 4, 2024

@rlarltj
기서님!
제가 오늘 �api 테스트를 하려고 카카오 로그인을 했는데 에러가 발생했습니다. 혹시 뭐가 잘못된 걸까요??
image
이런 에러가 떴어요. id값은 가져오는데 email 값을 못가져오더라구요. 그래서 홈페이지 가서 확인해보니
image
저번에 얘기했었던 사업자 등록 문제때문에 email�, name 정보를 못가져와서 에러가 발생한 것 같은데 혹시 맞을까요?? 그렇담 예외처리를 해줘야 할 것 같아서요.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants