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

Refactor/Socket Stability #371

Merged
merged 3 commits into from
Nov 28, 2024
Merged

Refactor/Socket Stability #371

merged 3 commits into from
Nov 28, 2024

Conversation

8954sood
Copy link
Member

개요 (필수)

  • 현재 서버에서 구독을 통해 제공하는 에러 예외처리는 현재 로직으로도 충분히 처리가 가능하다고 판단했습니다. 그래서 해당 기능을 추가하지 않았습니다.
  • 소켓의 안정성을 높였습니다.

이슈 번호

@8954sood
refactor: Socket Stability
346c97e 
socket의 연결, 연결해제 주기를 수정하여 더욱 안정적인 채팅이 가능해졌습니다.
@8954sood
chore: Edit Failed Send Connect Socket
d8ff680
@8954sood 8954sood added Refactor 코드 리펙토링 Feature 새로운 기능 개발 labels Nov 28, 2024
@8954sood 8954sood added this to the Seugi 4st develop milestone Nov 28, 2024
@8954sood 8954sood requested a review from wnsgur1 November 28, 2024 11:15
@8954sood 8954sood self-assigned this Nov 28, 2024
wnsgur1
wnsgur1 requested changes Nov 28, 2024
View reviewed changes
Copy link
Collaborator

@wnsgur1 wnsgur1 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@8954sood 8954sood force-pushed the feature/370-socket-error-channel-subscribe branch from 500a4b8 to bb422d5 Compare November 28, 2024 11:29
@8954sood 8954sood requested a review from wnsgur1 November 28, 2024 11:29
@8954sood 8954sood merged commit 841e872 into develop Nov 28, 2024
1 check passed
@8954sood 8954sood deleted the feature/370-socket-error-channel-subscribe branch November 28, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wnsgur1 wnsgur1 wnsgur1 approved these changes

Assignees

@8954sood 8954sood

Labels
Feature 새로운 기능 개발 Refactor 코드 리펙토링
Projects
Seugi
Status: Done
Milestone
Seugi 4st develop
Development

Successfully merging this pull request may close these issues.

Create Socket Error Channel Subscribe
2 participants
@8954sood @wnsgur1