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

Feature :: 소켓 애러 핸들링 #333 #345

Merged
merged 6 commits into from
Nov 23, 2024
Merged

Feature :: 소켓 애러 핸들링 #333 #345

merged 6 commits into from
Nov 23, 2024

Conversation

yeseong0412
Copy link
Member

@yeseong0412 yeseong0412 commented Nov 21, 2024

#️⃣ 연관된 이슈

#333

📝 작업 내용

기존 소캣 로직 동작과정에서
오류 발생시 모두 소켓이 close 되는 상황이 있었습니다. 이를 소켓 핸들링을 통해 오류 메시지를 반환하게 했으며
비즈니스 오류시엔 소켓 연결이 안 끊어지도록 만들었습니다.

추가적으로 기존 해더에 유저 정보를 넘기는 방식으로 유저인증 기반을 사용하였지만 simp 기반의 시큐리티 기반으로 변경하였습니다.

관련사항 문서 확인 부탁드립니다.

📎 ETC

기타

King is back 🚀

Response 베이스 클래스와 인터페이스를 추가하고, ErrorResponse, BaseResponse 가 상속받도록 변경함
RabbitMQConfig 패키지 위치 변경 ErrorResponse Status 기본값 변경
SocketException 핸들러에 세션 제거 기능을 추가했습니다. 또한, runtime 예외 메시지를 표시하는 방식을 개선했습니다.
STOMP 웹소켓 오류 처리를 위한 StompErrorHandler 추가. StompWebSocketConfig에 에러 핸들러 설정 및 사용자 목적지 접두사 설정.
@yeseong0412 yeseong0412 added the Feat 개발사항 label Nov 21, 2024
@yeseong0412 yeseong0412 requested a review from seu1a November 21, 2024 11:46
@yeseong0412 yeseong0412 self-assigned this Nov 21, 2024
@yeseong0412 yeseong0412 changed the title Featrue :: 소켓 애러 핸들링 #333 Feature :: 소켓 애러 핸들링 #333 Nov 21, 2024
@yeseong0412 yeseong0412 added the Refactor 리팩토링 사항 label Nov 21, 2024
Copy link
Member

@seu1a seu1a left a comment

Choose a reason for hiding this comment

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

확인했습니다 👍

@yeseong0412 yeseong0412 merged commit 4e2c2c9 into main Nov 23, 2024
1 check passed
@yeseong0412 yeseong0412 deleted the featrue/#333 branch November 23, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feat 개발사항 Refactor 리팩토링 사항
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants