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.

Sign up for GitHub

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

Jump to bottom

[BE] 학생 인증 코드가 유효하지 않을 시 500이 아닌 400 에러를 반환한다. #447

Closed
xxeol2 opened this issue Sep 15, 2023 · 3 comments · Fixed by #622
Closed

[BE] 학생 인증 코드가 유효하지 않을 시 500이 아닌 400 에러를 반환한다. #447

xxeol2 opened this issue Sep 15, 2023 · 3 comments · Fixed by #622
Labels
BE 백엔드에 관련된 작업 🙋‍♀️ 제안 제안에 관한 작업

Comments

@xxeol2
Copy link
Member

xxeol2 commented Sep 15, 2023

✨ 세부 내용

현재 학생인증 코드가 숫자가 아니거나, 6자리가 아니면 500 에러가 발생합니다.
image
하지만 이는 사용자가 코드를 잘못 입력한것으로, 500에러가 아닌 400에러가 나는 것이 더 적절하다 생각하는데 어떻게 생각하시나요!

⏰ 예상 소요 시간

30분
@xxeol2 xxeol2 added BE 백엔드에 관련된 작업 🙋‍♀️ 제안 제안에 관한 작업 labels Sep 15, 2023
@seokjin8678
Copy link
Collaborator

도메인 계층의 입력 값에 대한 검증은 InternalServer를 던지고, 프레젠테이션 계층의 입력 값에 대한 검증은 BadRequest 예외를 던지게 하는 것은 어떤가요?

@xxeol2
Copy link
Member Author

xxeol2 commented Sep 26, 2023

cc. #448 (comment)

@xxeol2
Copy link
Member Author

xxeol2 commented Oct 1, 2023

@seokjin8678 seokjin8678 linked a pull request Nov 16, 2023 that will close this issue
[BE] fix: 학생 인증 코드가 유효하지 않을 시 500이 아닌 400 에러를 반환 (#447) #622
Merged
@seokjin8678 seokjin8678 mentioned this issue Nov 16, 2023
Merged
seokjin8678 added a commit that referenced this issue Dec 21, 2023
@seokjin8678
[BE] fix: 학생 인증 코드가 유효하지 않을 시 500이 아닌 400 에러를 반환 (#447) (#622)
6d48b30 
* refactor: UnexpectedExcepton -> ValidException 변경

- 사용자 입력에 영향을 받는 생성자 이므로 400 예외를 던지기 위함

* test: 테스트 코드 반영
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-festa-go Dec 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BE 백엔드에 관련된 작업 🙋‍♀️ 제안 제안에 관한 작업
Projects
2023-festa-go
Status: Done
Milestone
No milestone
2 participants
@xxeol2 @seokjin8678