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

[Bug] /intra-auth 응답코드 #162

Open
Yaminyam opened this issue Apr 4, 2022 · 6 comments
Open

[Bug] /intra-auth 응답코드 #162

Yaminyam opened this issue Apr 4, 2022 · 6 comments
Assignees

Comments

@Yaminyam
Copy link
Member

Yaminyam commented Apr 4, 2022

오류 사진 및 설명

image

발생 위치

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

해결시도

@Yaminyam Yaminyam changed the title [Bug] [Bug] /intra-auth 응답코드 Apr 4, 2022
@Yaminyam
Copy link
Member Author

Yaminyam commented Apr 6, 2022

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

@Skyrich2000
Copy link
Member

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

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

@Yaminyam
Copy link
Member Author

Yaminyam commented Apr 6, 2022

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

@Skyrich2000
Copy link
Member

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

@Yaminyam
Copy link
Member Author

Yaminyam commented Apr 6, 2022

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

@Skyrich2000
Copy link
Member

https://stackoverflow.com/questions/58824401/disable-status-201-for-all-posts-in-nestjs 코드자체는 금방 수정할수있겠네요

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

No branches or pull requests

2 participants