Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to new standards? #18

Open
Neograph734 opened this issue Sep 9, 2015 · 3 comments
Open

Update to new standards? #18

Neograph734 opened this issue Sep 9, 2015 · 3 comments

Comments

@Neograph734
Copy link

In Firefox JavaScript Console there are messages like:

RTCIceServer.url is deprecated! Use urls instead.

And

Mandatory/optional in createOffer options is deprecated! Use {"offerToReceiveAudio":true,"offerToReceiveVideo":true} instead (note the case difference)!

The messages don't really bother me, but I suppose they might be dropped at some point. So should these be updated?

@albertosouza
Copy link

Now this dont works in FF 43:
console error:

Mandatory/optional in createOffer options no longer works! Use {"offerToReceiveAudio":true,"offerToReceiveVideo":true} instead (note the case difference)!
ICE failed, see about:webrtc for more details

@nathanoehlman
Copy link
Contributor

Hi @albertosouza - I take it you are using the build at //cdn.jsdelivr.net/rtc/latest/rtc.min.js? If so, unfortunately that build hasn't been rebuilt in a while to take advantage of the updates to some of the other rtc.io packages, which include the correct use of FF offer constraints (although it still needs to use RTCIceServer.urls).

I've deployed a build a new 3.4.0 release now, but we're just waiting on the jsdelvr CDN to pick it up.

Additionally, as a note, I'll push out a 4.0.0 series release in a couple of days - rtc is still using the ^4.5 releases of rtc-quickconnect (the main underlying package), while this has moved onto a ^5 version.

@albertosouza
Copy link

ok @nathanoehlman i think i tested a old version.
You need help with something in this project?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants