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

refactor: POST 요청 응답 상태코드 변경 (200 → 201) 완료 #593

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

limehee
Copy link
Collaborator

@limehee limehee commented Oct 19, 2024

Summary

#592

POST 요청의 응답 상태 코드를 변경하는 필터를 추가했습니다.
이 필터는 /api/* 경로에 대한 POST 요청의 응답 상태 코드가 200일 경우 이를 201로 수정합니다.

Tasks

  • /api/* 경로의 모든 POST 요청을 대상으로 HttpServletResponse의 상태 코드가 200일 경우 201로 변경하도록 하는 서블릿 필터 추가

Log

POST

INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/recruitments POST 201 79ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/books POST 201 27ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/awards POST 201 34ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/awards POST 201 4ms

POST 외 메소드

INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/awards?page=0&size=20&sortBy=awardDate&sortDirection=desc GET 200 110ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/members/superuser PATCH 200 20ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/awards?page=0&size=20&sortBy=awardDate&sortDirection=desc GET 200 9ms
INFO  page.clab.api.global.util.ApiLogger - [0:0:0:0:0:0:0:1:test] /api/v1/awards/2 DELETE 200 13ms

Screenshot

image

@limehee limehee added the 🔨 Refactor 코드 수정 및 개선 label Oct 19, 2024
@limehee limehee requested a review from gwansikk October 19, 2024 14:47
@limehee limehee self-assigned this Oct 19, 2024
@limehee limehee linked an issue Oct 19, 2024 that may be closed by this pull request
Copy link
Member

@gwansikk gwansikk left a comment

Choose a reason for hiding this comment

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

점진적으로 상태 코드를 명확하게 적용해봅시다 👍👍👍

@mingmingmon @SongJaeHoonn 응답에 대한 HTTP 상태 코드를 좀 더 명확하게 하기로 결정했어요(모니터링과 통계에 정확성을 높이고자). 회의 없이 관희님이랑 간단하게 얘기하고 증명해본 PR입니다. 앞으로 점진적으로 적용해 나갈 예정이니 참고 부탁드려요~~

@mingmingmon
Copy link
Collaborator

점진적으로 상태 코드를 명확하게 적용해봅시다 👍👍👍

@mingmingmon @SongJaeHoonn 응답에 대한 HTTP 상태 코드를 좀 더 명확하게 하기로 결정했어요(모니터링과 통계에 정확성을 높이고자). 회의 없이 관희님이랑 간단하게 얘기하고 증명해본 PR입니다. 앞으로 점진적으로 적용해 나갈 예정이니 참고 부탁드려요~~

넵! created 의미를 잘 나타낼 수 있도록 201로 수정해주신거군요. 확인했습니다!

@SongJaeHoonn
Copy link
Contributor

200 -> 201 확인했습니다!

@limehee limehee merged commit eaa6c9c into develop Oct 23, 2024
2 checks passed
@limehee limehee deleted the refactor/#592 branch October 23, 2024 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 Refactor 코드 수정 및 개선
Projects
None yet
Development

Successfully merging this pull request may close these issues.

POST 요청 응답 상태코드 변경 (200 → 201)
4 participants