Skip to content
This repository has been archived by the owner on May 19, 2024. It is now read-only.

[WEAV-70] 소셜 로그인 API, 회원가입 필요 응답 상태코드 401 명세 #22

Merged
merged 2 commits into from
Jan 27, 2024

Conversation

waterfogSW
Copy link
Member

@waterfogSW waterfogSW commented Jan 27, 2024

클라이언트측 요청 사항 반영이에요

문장훈 — 어제 오후 10:29
@서버 
소셜 로그인 API 호출 시 회원가입 되어있는 경우만 테스트 가능해서
registerToken이 오는 경우는 어떻게 구분할 수 있는지 필요합니다. 
ex. 상태코드 

회원가입 필요한 경우 Http 기본 상태코드중 401로 응답하도록 처리했어요.

@waterfogSW waterfogSW requested a review from dojinyou as a code owner January 27, 2024 05:31
@waterfogSW waterfogSW self-assigned this Jan 27, 2024
@waterfogSW waterfogSW added the feat New feature or request label Jan 27, 2024
Comment on lines +18 to +23
): ResponseEntity<SocialLoginResponse> {
val success = SocialLoginResponse.Success(
accessToken = "test_access_token",
refreshToken = "test_refresh_token",
)
return ResponseEntity.ok(success)
Copy link
Member Author

Choose a reason for hiding this comment

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

UseCase구현 완료되면 의존성 추가하면서 회원가입 필요한 경우 에대한 분기처리 구현 예정이에요

@waterfogSW waterfogSW requested a review from dojinyou January 27, 2024 07:04
@waterfogSW waterfogSW changed the title [WEAV-70] 소셜 로그인 API, 회원가입 필요 응답 상태코드 202 명세 [WEAV-70] 소셜 로그인 API, 회원가입 필요 응답 상태코드 401 명세 Jan 27, 2024
Copy link

sonarcloud bot commented Jan 27, 2024

@waterfogSW waterfogSW enabled auto-merge (squash) January 27, 2024 07:07
@waterfogSW waterfogSW merged commit a5accfe into main Jan 27, 2024
3 checks passed
@waterfogSW waterfogSW deleted the feat/WEAV-70 branch January 27, 2024 07:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feat New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants