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

Test/implement reconnect #23

Closed muellni closed 7 years ago

muellni commented 7 years ago

Make the testclient reconnect to the server, make it buffer ICE packages without server connection and test switching from LAN to Wifi

micheljung commented 7 years ago

Why would we buffer ICE packages? (maybe I don't understand what those packages are). If those are just game packages, I see no need to buffer them. If those are ICE specific messages, I'm also not sure they need to be buffered (ICE would just send them again, wouldn't it?).

muellni commented 7 years ago

I'm currently trying to figure out how to do reconnect with webrtc. I'm not sure if the testclient has to buffer the signalling packages.

muellni commented 7 years ago

I'm now creating a whole new rtcpeerconnection because ice-reconnect is currently unsupported by the node-webrtc bindings. This works most of the time.