ZeroCho / nodejs-book

Node.js교과서 소스 코드
808 stars 646 forks source link

12장 socket.js의 socket.adapter 관련 issue #145

Open jrucl2d opened 3 years ago

jrucl2d commented 3 years ago

Zerocho님 책으로 공부중인 학생입니다. 현재 12장 gifchat 부분을 공부중인데 socket.on('discconection' 내부에서 const currentRoom = socket.adapter.rooms[roomId]; const userCount = currentRoom ? currentRoom.length : 0; 위 두 부분 값이 정상적으로 반영되지 않는 것을 확인하여 직접 console.log를 찍어보니 각각 객체와 배열이 아닌 socket.adapter.rooms는 Map으로, 그 안의 socket.id들은 Set으로 되어있었습니다. const currentRoom = socket.adapter.rooms.get(roomId); const userCount = currentRoom ? currentRoom.size : 0; 로 변경한 결과 정상적으로 작동하는 것을 확인했는데 socket.io의 업데이트로 인한 변경인가 싶어서 이슈 남깁니다!

ZeroCho commented 3 years ago

안녕하세요. socket.io 3버전이 출시됨에 따라 변경된 사항인 것 같습니다. 제보 감사합니다.