Bug 1247619 - add console warning about OAuth for STUN not yet supported. r=jesup,smaug

MozReview-Commit-ID: L0nViIWiIDn

--HG--
extra : rebase_source : 3d463e85c811426384a758dca3b0a82370c22099
This commit is contained in:
Jan-Ivar Bruaroey 2016-02-25 00:24:16 -05:00
parent 2f131341dd
commit c4ea91fa82
3 changed files with 17 additions and 4 deletions

View File

@ -586,6 +586,13 @@ RTCPeerConnection.prototype = {
throw new this._win.DOMException(msg + " - missing credential: " + urlStr,
"InvalidAccessError");
}
if (server.credentialType != "password") {
this.logWarning("RTCConfiguration TURN credentialType \""+
server.credentialType +
"\" is not yet implemented. Treating as password."+
" https://bugzil.la/1247616",
null, 0);
}
}
else if (!(url.scheme in { stun:1, stuns:1 })) {
throw new this._win.DOMException(msg + " - improper scheme: " + url.scheme,

View File

@ -7,11 +7,17 @@
* http://dev.w3.org/2011/webrtc/editor/webrtc.html#idl-def-RTCConfiguration
*/
enum RTCIceCredentialType {
"password",
"token"
};
dictionary RTCIceServer {
(DOMString or sequence<DOMString>) urls;
DOMString url; //deprecated
DOMString? credential = null;
DOMString? username = null;
DOMString username;
DOMString credential;
RTCIceCredentialType credentialType = "password";
};
enum RTCIceTransportPolicy {

View File

@ -663,8 +663,8 @@ PeerConnectionConfiguration::AddIceServer(const RTCIceServer &aServer)
port = (isStuns || isTurns)? 5349 : 3478;
if (isTurn || isTurns) {
NS_ConvertUTF16toUTF8 credential(aServer.mCredential);
NS_ConvertUTF16toUTF8 username(aServer.mUsername);
NS_ConvertUTF16toUTF8 credential(aServer.mCredential.Value());
NS_ConvertUTF16toUTF8 username(aServer.mUsername.Value());
if (!addTurnServer(host.get(), port,
username.get(),