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

[FEAT/#4] P2P 연결을 위한 클래스들을 구현합니다. (WebSocket, Signaling, WebRTC, Connection) #16

Merged
merged 22 commits into from
Nov 12, 2024

Conversation

youn9k
Copy link
Collaborator

@youn9k youn9k commented Nov 12, 2024

🤔 배경

P2P 연결을 위한 기능을 구현해야 했습니다.

📃 작업 내역

시그널링 과정을 진행하기 위해 필요한 클래스와 타입을 구현했습니다.

타입

  • SessionDescription(SDP) 타입 구현
  • IceCandidate 타입 구현
  • SignalingMessage 타입 구현

클래스 및 프로토콜

  • WebSocketClientWebSocketClientDelegate 구현
  • SignalingCilentSignalingClientDelegate 구현
  • WebRTCClient WebRTCClientDelegate구현
  • ConnectionClient 구현

✅ 리뷰 노트

살려주세요.

🎨 스크린샷

도메인 작업만 진행해 스크린샷이 없습니다ㅜ

🚀 테스트 방법

URLProtocol을 이용해 소켓 통신 요청을 가로채는 Mock객체를 만들어 테스트를 진행하려했습니다.

XCTest 객체를 URLSessionWebSocketDelegate 채택해 콜백을 받으려 했지만, 호출되지 않는 문제가 있었고, 3시간 이상 트러블 슈팅을 해봤습니다.

하지만 성공하지 못했고, 작업 우선순위에 따라 다른 작업을 진행했습니다. 🥲

youn9k and others added 18 commits November 7, 2024 13:31
- secrets 추적 해제

Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
- BASE_URL 추가
- WebSocketClient (일단)구현

Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
- http 서버 대응

Co-Authored-By: Youngkyu Song <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
Co-Authored-By: seuhong <[email protected]>
Co-Authored-By: Kiyoung <[email protected]>
@youn9k youn9k added the ✨ feat 새로운 기능 추가 label Nov 12, 2024
@youn9k youn9k requested review from 0Hooni and hsw1920 November 12, 2024 06:03
@youn9k youn9k linked an issue Nov 12, 2024 that may be closed by this pull request
1 task
Copy link
Collaborator

@0Hooni 0Hooni left a comment

Choose a reason for hiding this comment

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

LGTVOLEDLGTM

@0Hooni 0Hooni self-requested a review November 12, 2024 07:13
Copy link
Collaborator

@0Hooni 0Hooni 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

@hsw1920 hsw1920 left a comment

Choose a reason for hiding this comment

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

클라이언트들의 docs가 절실합니다.

@youn9k youn9k merged commit 145554e into develop Nov 12, 2024
@youn9k youn9k deleted the feature/#4-session-connect branch November 12, 2024 08:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 새로운 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

세션 연결이 된다
4 participants