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

[BE] 소켓 연결 끊기는 현상 수정 #106

Merged
merged 2 commits into from
Nov 14, 2024
Merged

[BE] 소켓 연결 끊기는 현상 수정 #106

merged 2 commits into from
Nov 14, 2024

Conversation

sieunie
Copy link
Collaborator

@sieunie sieunie commented Nov 14, 2024

✅ 주요 작업

  • pong 데이터 한투 서버로 보내서 소켓 연결 끊김 현상 해결 (아침에 일어나면 소켓 안되는 현상 아마 해결...)

💭 고민과 해결과정

PINGPONG
한투에서 PINGPONG 데이터 보내서 자동으로 소켓 연결 끊김 해결해주는 줄 알았는데, 한투 서버에서는 PING만 해주고 우리 서버에서 한투 서버로 PONG까지 해줘야 연결이 유지가 되는 방식이었다.

image

위 이미지 참고해서 장 마감 시에 PINGPONG 데이터가 들어오면 pong 응답을 해주도록 아래와 같이 수정했다.
아직 장 마감이 안돼서 테스트는 못해본 상황이다.

        if (json.header.tr_id === 'PINGPONG') this.socket.pong(json);

장 마감되면 테스트 해보고 머지하겠습니다.

@sieunie sieunie added BE 백엔드 BUGFIX 버그 픽스 labels Nov 14, 2024
@sieunie sieunie requested review from uuuo3o and jinddings November 14, 2024 04:46
@sieunie sieunie self-assigned this Nov 14, 2024
@@ -48,6 +48,7 @@ export class BaseSocketService implements OnModuleInit {
`한국투자증권 웹소켓 연결: ${json.body.msg1}`,
json.header.tr_id,
);
if (json.header.tr_id === 'PINGPONG') this.socket.pong(json);
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 여기서 pong 이라는 건 직접 만들어주신건가요?? pong 이 무슨 역할하는지 궁금한데 인터넷에 검색해도 딱히 나오는게 없고 코드에서도 전체 검색했는데도 잘못 검색한건지 찾을 수가 없네요...?? 또, json은 왔던 걸 그대로 보내주는건가요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

pong은 웹소켓 자체에서 제공하는 메소드이구, 방금 대충 찾아봤는데 ping/pong을 보낼 때는 일반 프레임이랑 다른 프레임을 쓰는 것 같더라고요? 그래서 pong 프레임으로 데이터 보내도록 해주는 내부 메소드인 것 같습니다.
pong 데이터는 뭐 보내야할 지 모르겠어서 한투 파이썬 라이브러리 뒤져봤는데 거기서도 그냥 왔던거 그대로 보내주는 것 같아서 이렇게 구현해두었어용

Copy link
Collaborator

Choose a reason for hiding this comment

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

아항 ㅎㅎㅎ 감사합니다.
https://m.blog.naver.com/techshare/223319517781
궁금해서 막 찾아봤는데 이런 글도 있네요 ㅋㅋㅋㅋㅋ!! 일단 장마감까지 한번 기다려봅시다.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

옹 감사합니다!! 저도 더 찾아봐야겠어요

@sieunie sieunie merged commit 7e209bc into back/main Nov 14, 2024
2 checks passed
@sieunie sieunie deleted the fix/socket branch November 15, 2024 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 BUGFIX 버그 픽스
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants