Skip to content

Conversation

@ShipFriend0516
Copy link
Member

Note

서버측 세션 리팩토링에 대응해서 클라이언트 코드 수정했습니다.

  • 작성자: 서정우
  • 작성 날짜: 2024.11.27

✅ 체크리스트

  • 코드가 정상적으로 작동하는지 확인했습니다.
  • 주요 변경사항에 대한 설명을 작성했습니다.
  • 코드 스타일 가이드에 따라 코드를 작성했습니다.

🧩 작업 내용

  • 변경된 웹소켓 이벤트명으로 변경
  • 이벤트명 분리해서 관리
  • 기존 타입 수정

📝 작업 상세 내역

화상통화에 다른 유저가 들어올 때 내 화면이 하나 더생기는 이슈

  • 이 이슈는 유저가 처음 세션에 입장할 때 받아야하는 room_join 응답 문제였다.
  • 해당 응답이 세션에 처음 입장한 유저만이 받는게 아닌 전체 유저가 한 번 더 받아서 생긴 문제였다.
  • 세션 입장한 한명만 받아올 수 있도록 수정해서 해결했다.

ICE CANDIDATE 오류

  • 로그에는 피어 커넥션이 이루어졌고, 에러가 없다고 떴지만 화상통화가 서로 안되는 모습이 생겼다.
  • 상대방의 스트림은 받아왔다고 로그가 있었다.
  • chrome://webrtc-internals/ 페이지에서 ice 상태가 not connect가 뜨는 것을 보고 ice candidate 문제임을 파악했다.
  • 실제로 ice candidate 코드에 문제가 있었고, 수정 후에 정상 작동했다.

🐥 리뷰 받고 싶은 포인트(선택)

프론트

이벤트 명을 SESSION_EMIT_EVENTS , SESSION_LISTEN_EVENTS로 분리했습니다. 시그널링 서버도 비슷하게 수정했습니다.
프론트에서 보내야하는 이벤트는 SESSION_EMIT_EVENTS.CREATE 이런 방식으로 사용할 수 있습니다. 한 번 보고 숙지해보는게 좋을 것 같습니다.

백엔드

명세서가 큰 폭으로 변경될 경우에 이름이 AAA에서 BBB로 변경되었습니다. 정도로만 명시해주시면 감사하겠습니다.

blu3fishez and others added 25 commits November 26, 2024 16:45
- 최소 1개, 최대 3개 의 string 배열을 받도록 수정
- connectionList => connectionMap
- connectionMap 에서 자신의 연결정보 삭제하고 전송
Copy link
Member

@yiseungyun yiseungyun left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@blu3fishez blu3fishez left a comment

Choose a reason for hiding this comment

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

엘지티엠

@ShipFriend0516
Copy link
Member Author

@yiseungyun 듬뮴밈 이벤트명이랑 응답형식이 바뀐게 있어서 테스트 코드가 아마 작동을 안할 것 같습니다...ㅠㅠ

Copy link
Collaborator

@twalla26 twalla26 left a comment

Choose a reason for hiding this comment

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

LGTM! 고생하셨습니다~! 👍

@ShipFriend0516 ShipFriend0516 merged commit 5191c8d into boostcampwm-2024:dev-fe Nov 27, 2024
@ShipFriend0516 ShipFriend0516 deleted the refactor/session-page branch November 27, 2024 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎁 Refactoring 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants