search

FAForever / ice-adapter

Deprecated C++ ICE adapter - please use java-ice-adapter instead!
https://github.com/FAForever/java-ice-adapter
GNU General Public License v3.0
11 stars 10 forks source link

Fix ICE State #16

Closed IDragonfire closed 7 years ago

IDragonfire commented 7 years ago

Currently the ICE State flow looks a little bit broken ...

How to Reproduce

  1. One Test Client host the came
  2. Alice/Bob/Charlie joins the game

2 Player Scenario

image

3 Player Scenario

image

4 Player Sceanario

image

Notes

https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/iceConnectionState

IDragonfire commented 7 years ago

https://bugs.chromium.org/p/webrtc/issues/detail?id=1414#c17

muellni commented 7 years ago

I think the testclient works better now. I saw both ICE states "connected" and "complete" both led to successful connections.