Skip to content

Feature/56 modify res status#62

Open
sua-kim wants to merge 5 commits into
developfrom
feature/56-modify-res-status
Open

Feature/56 modify res status#62
sua-kim wants to merge 5 commits into
developfrom
feature/56-modify-res-status

Conversation

@sua-kim
Copy link
Copy Markdown
Collaborator

@sua-kim sua-kim commented Aug 13, 2021

No description provided.

@sua-kim sua-kim added the enhancement New feature or request label Aug 13, 2021
@sua-kim sua-kim requested a review from JangBomi August 13, 2021 17:17
@sua-kim sua-kim self-assigned this Aug 13, 2021
@sua-kim sua-kim linked an issue Aug 13, 2021 that may be closed by this pull request
@sua-kim
Copy link
Copy Markdown
Collaborator Author

sua-kim commented Aug 13, 2021

  • client에서 success 여부와 message 정보는 필요 없다고 해서 응답코드 send로 통일해서 수정했어요!
  • delete 라우터에서 id가 DB에 없음에도 오류 캐치가 안되는 현상 발생
    --> findOne으로 예외 처리했으나 추가 확인 필요함

<http 응답코드 정리>

(1) 성공
200 : get 요청 성공 및 데이터 전송
201: post 요청 성공 및 데이터 전송
204: PUT, POST, DELETE 요청의 경우 성공은 했지만 전송할 데이터가 없는 경우
--> res.json('success':true)일 경우 res.status(204).end()로 전면 수정

(2) 클라이언트 오류
400: 요청 오류 (요청한 id가 서버에 존재하지 않는 경우)
403: 카카오 인증 오류 (authorization 토큰 오류)
409: 회원가입 시 사용자 정보가 중복될 경우

(3) 서버 오류
500: 서버 내부 오류 (특정할 수 없는 경우)
501: 특정할 수 있는 서버 오류 (res.send로 메세지 전송)
503: 서버에 정보가 없을 때, 즉 null 값을 리턴할 경우

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

response status 정리

1 participant