Skip to content
This repository was archived by the owner on Mar 22, 2022. It is now read-only.

Fix data channel before connect#11

Merged
jehumb-havok merged 4 commits intodevfrom
bugs/data_channel_before_connect
Jul 9, 2019
Merged

Fix data channel before connect#11
jehumb-havok merged 4 commits intodevfrom
bugs/data_channel_before_connect

Conversation

@jehumb-havok
Copy link
Copy Markdown
Member

Ensure that data channel can be created after the peer connection is initialized but before starting the SDP connection handshake, so that the later can integrate SCTP handshake. Otherwise there is no way to use data channels.

Return some error codes from C/C++ and transform those into exceptions in C#. This fixes a NULL pointer dereferencing in TestAppUWP, which only handles exceptions.

Ensure that the SCTP negotiating is only preventing
`AddDataChannelAsync` after the connection started, and not before,
which is the only way to effectively ensure it occurs.
@jehumb-havok jehumb-havok added the bug Something isn't working label Jul 8, 2019
@jehumb-havok jehumb-havok added this to the Initial public preview milestone Jul 8, 2019
Comment thread libs/Microsoft.MixedReality.WebRTC/PeerConnection.cs
Comment thread libs/Microsoft.MixedReality.WebRTC/PeerConnection.cs
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants