diafygi / webrtc-ips

Demo: https://diafygi.github.io/webrtc-ips/
MIT License
3.44k stars 598 forks source link

Not working on Chrome72 #44

Open EtherDream opened 5 years ago

blowing-in-the-wind commented 5 years ago

@EtherDream

var servers = {iceServers: [{urls: "stun:stun.services.mozilla.com"}], sdpSemantics:'plan-b'};

Refer:

Justin2k commented 5 years ago

I see that Phase 4 of the unified plan is to remove support for 'plan-b': https://webrtc.org/web-apis/chrome/unified-plan/

Has anyone got a fix when using 'unified-plan' rather than 'plan-b'?

Justin2k commented 5 years ago

I've managed to figure this out. It was just down to the createDataChannel method needing to have a value for it's Label parameter rather than being blank.

I've suggested a new pull request that applies this change.

vitalets commented 5 years ago

I'm maintaining a fork of webrtc-ips that currently works in Chrome 72 in my tests. It also supports passing custom urls in params. Maybe it will help.

nebi-frame commented 5 years ago

@vitalets what about safari?