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

채팅 기능의 예외 상황 처리 및 발생할 수 있는 예외 클래스 핸들링 #212

Merged
merged 4 commits into from
Aug 5, 2024

Conversation

Mingyum-Kim
Copy link
Contributor

@Mingyum-Kim Mingyum-Kim commented Aug 4, 2024

PR의 목적이 무엇인가요?

  • 채팅 기능을 구현하는 과정에서 발생할 수 있는 오류를 처리함.
  • 클라이언트에서 잘못된 입력 시 스프링에서 던지는 예외를 핸들링함.
  • 추가적인 기능 상 예외를 처리함.

이슈 ID는 무엇인가요?

설명

image

만약 @RequestParam을 사용하였다면 기본적으로 required = true옵션이 걸려있습니다.
이 경우 MissingServletRequestParameterException 예외 클래스가 던져지는데 이를 핸들링하여 우리의 에러 응답을 반환하도록 하였어요.

그리고 채팅 조회 시 파라미터의 예외를 처리하였습니다.

모임에 참여하지 않는 회원일 때 채팅을 보내는 / 조회하는 예외를 처리함.

질문 혹은 공유 사항 (Optional)

@Mingyum-Kim Mingyum-Kim added BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Aug 4, 2024
@Mingyum-Kim Mingyum-Kim self-assigned this Aug 4, 2024
Copy link
Contributor

@pricelees pricelees left a comment

Choose a reason for hiding this comment

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

굿~! 나머지 requestparam 예외도 처리해봅시다요!

Copy link
Contributor

@hoyeonyy hoyeonyy left a comment

Choose a reason for hiding this comment

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

너무 좋아요 제니님~!

@Mingyum-Kim Mingyum-Kim merged commit d305bd1 into develop-backend Aug 5, 2024
1 check passed
@Mingyum-Kim Mingyum-Kim deleted the feature/#211 branch August 5, 2024 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants