42-world / 42world-Backend

42서울 재학생과 졸업생 모두를 아우르는 커뮤니티 | 42world 백엔드 저장소

Home Page:https://42world.kr

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Bug] /intra-auth 응답코드

Yaminyam opened this issue · comments

오류 사진 및 설명

image

발생 위치

[POST] /intra-auth 에서 이메일 전송을 성공했을때 응답코드가 201인데 200성공으로 명세되어있습니다.

해결시도

다른 응답들이 모두 200 OK로 결과가 날아오니 swagger 명세를 수정하기보다 응답을 201에서 200코드로 수정하는것이 좋아보입니다.

commented

이게 뭔가 객체를 생성한 경우에는 nest.js 에서 자동으로 201로 처리하더라구요. 200으로 고정하는 방법을 찾아봐야할거같습니다.

intra-auth 말고도 다른 POST 요청에 대해서 swagger에는 200으로 되어있지만 실제로는 201로 되는 api들이 종종 있습니다.
게시글 생성이나, 댓글 생성같은경우는 의미상 201이 더 맞는거같아서 코드를 고치기보다는 swagger를 수정하는게 좋을거같네요

그러게요 방금 다시 확인해봤는데 post하는 api들 전부 next.js에서 201로 처리해주는것 같네요.
200으로 고정하기보다는 201이 자세한 응답이니 응답을 고치기 보다 후에 말씀하신것처럼 swagger를 수정하는 방향이 좋아보입니다.

commented

POST /intra-auth 는 의미상 200이 나을지 201이 나을지 조금 고민되네요...

intra-auth는 작동방식을 제가 잘 몰라서...
일단 그러면 다른 확실한 친구들 swagger 명세 수정해두겠습니다.