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.

Sign up for GitHub

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

Jump to bottom

JsSIP not compatible with Firefox when using TURN with credentials #176

Closed
pdunkley opened this issue Nov 18, 2013 · 1 comment
Closed

JsSIP not compatible with Firefox when using TURN with credentials #176

pdunkley opened this issue Nov 18, 2013 · 1 comment
Labels
interop WebRTC
Milestone
0.3.X

Comments

@pdunkley
Copy link

I am using JsSIP at RTCWebIt 2 with my own TURN server with authentication.

JsSIP uses the deprecated (but still supported by Chrome) URI form for TURN. This deprecated form doesn't work with Firefox, but the new form works with both Chrome and Firefox.

To fix this just change:
'url': scheme + ':' + server.username + '@' + url.substr(scheme.length+1),
'credential': server.password

to:
'url': scheme + ':' + url.substr(scheme.length+1),
'username': server.username,
'credential': server.password

in RTCSession/RTCMediaHandler.js:init()

Other than this JsSIP seems to work great with both Firefox and Chrome.
@jmillan
Copy link
Member

jmillan commented Nov 19, 2013

Hi Peter,

We will also allow multiple URLs for the same RTCIceServer.

Thanks a lot

jmillan added a commit that referenced this issue Nov 19, 2013
@jmillan
Fix #176. Update to the latest IceServer definition
8f62d3a 
Old:

dictionary RTCIceServer {
    DOMString  url;
    DOMString? credential;
};

New:

dictionary RTCIceServer {
    (DOMString or sequence<DOMString>) urls;
    DOMString?                         username = null;
    DOMString?                         credential;
};

Note: Current Firefox Nightly "28.0a1 (2013-11-18)" Does not support multiple 'urls'
@dpocock dpocock mentioned this issue Jan 9, 2014
Closed
mEkblom added a commit to mEkblom/JsSIP that referenced this issue Apr 14, 2014
@mEkblom
Allow empty list of STUN servers (broken by fix versatica#176 in JsSIP)
26c05ba
@IlyaRu IlyaRu mentioned this issue Oct 30, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
interop WebRTC
Milestone
0.3.X
Development

No branches or pull requests
2 participants
@jmillan @pdunkley