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] 소켓 로그 추가 #96

Merged
merged 1 commit into from
Nov 13, 2024
Merged

[BE] 소켓 로그 추가 #96

merged 1 commit into from
Nov 13, 2024

Conversation

sieunie
Copy link
Collaborator

@sieunie sieunie commented Nov 13, 2024

✅ 주요 작업

  • 한국투자 소켓 연결 시 로깅
  • 5분마다 소켓 데이터 들어올 때 로깅
  • 5분마다 클라이언트로 소켓 데이터 보낼 때 로깅

image

@sieunie sieunie added BE 백엔드 REFACTORING 코드 리팩터링 labels Nov 13, 2024
@sieunie sieunie self-assigned this Nov 13, 2024
Copy link
Collaborator

@uuuo3o uuuo3o left a comment

Choose a reason for hiding this comment

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

간단한 로그 추가 좋네요ㅎㅎ!
실패/연결 끊김 등의 상황에서 재연결 로직은 천천히 한번 생각해봅시다! 고생하셨어용

if (data.length < 2) {
const json = JSON.parse(data[0]);
this.logger.log(
`한국투자증권 웹소켓 연결: ${json.body.msg1}`,
Copy link
Collaborator

Choose a reason for hiding this comment

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

🟢 지금 당장은 연결, 수신 성공 로그만 남겨도 괜찮네요!

@sieunie sieunie merged commit cc95624 into back/main Nov 13, 2024
2 checks passed
@sieunie sieunie deleted the fix/socket-log 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 백엔드 REFACTORING 코드 리팩터링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants