-
Notifications
You must be signed in to change notification settings - Fork 4
[Feat]signaling server 구현 #54
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
[Feat]signaling server 구현 #54
Conversation
|
PR을 너무 잘 써주셔서 이해가 잘 됐습니다. 예상보다 빠르게 구현이 되었네요! nestjs 코드리뷰는 처음인데 덕분에 nestjs 문법도 알 수 있어서 좋았습니다 👍👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR 상세하게 써주시고, 결과 스크린 샷까지 넣어주셔서 잘 읽을 수 있었습니다. 👍
spec.ts 파일에 테스트하는 코드가 있어, 찾아보니 nest에서는 *.spec.ts로 테스트 파일 명을 정하는 걸 알게되었네요.
yiseungyun
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nest를 써보지 않았음에도 코드가 깔끔해서 읽기 쉬웠던거 같습니다. 수고 많으셨습니다. ☃️
blu3fishez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
일단은 간단한 예제를 배포를 해서 같이 테스트를 해봅시다!
이번에는 Approve 인데.. any 타입은 절대 쓰시면 안됩니다..ㅠㅠ
| handleConnection(socket: any) { | ||
| console.log(`Client connected: ${socket.id}`); | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any 타입 절대 쓰시면 안됩니다...!
socket으로 어떤 데이터가 오는지 모르는 경우 unknown 타입을 사용해서 타입가드로 내로잉하셔야할 거에요..!
관련 개념 찾아보시면 좋을 것 같습니다.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any와 unknown 타입!! 학습하겠습니다.
| private users: { [key: string]: User[] } = {}; | ||
| private socketToRoom: { [key: string]: string } = {}; | ||
| private maximum = 5; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
임시로 외부 변수 사용하신거 일단은 좋습니다~ 개념이 어렵다보니 일단 심플하게 그냥 하시는 게 좋죠!
이걸 같이 Redis 로 뺄 방법을 같이 공부해봅시다!!
백엔드 signaling server 구현
관련 이슈 번호
✅ 체크리스트
🧩 작업 내용
📝 작업 상세 내역
웹소켓 연결 설정 및 해제 기능
signaling server 구현
📌 테스트 및 검증 결과
웹소켓 연결 설정 및 해제 기능
signaling server 구현
방 참여, 퇴장 처리
offer 처리
💬 다음 작업 또는 논의 사항
📎 참고 자료