Skip to content
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

expose channels (PC), allowing KITE test #340

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

expose channels (PC), allowing KITE test #340

wants to merge 1 commit into from

Conversation

Piasy
Copy link

@Piasy Piasy commented Apr 1, 2020

No description provided.

@Piasy Piasy requested a review from jianjunz as a code owner April 1, 2020 13:06
@jianjunz
Copy link
Collaborator

jianjunz commented Apr 1, 2020

We have a proposal to expose RTCRtpTransceivers, and OWT SDK will take care of the relationship between OWT objects and WebRTC objects. But exposing PeerConnection could cause unexpected calls PeerConnections, and break the relationship by accident. We still need to better design this API, or expose it for testing only, perhaps behind a flag.

@jianjunz
Copy link
Collaborator

jianjunz commented Apr 1, 2020

@taste1981 is investigating KITE integration as well. He may have comments on this.

@jianjunz jianjunz requested a review from taste1981 April 1, 2020 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jianjunz jianjunz Awaiting requested review from jianjunz jianjunz is a code owner

@taste1981 taste1981 Awaiting requested review from taste1981 taste1981 is a code owner

At least 6 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Piasy @jianjunz