From 669d234ebd8bcf07adbbc9a6ab422e52c4dc15fe Mon Sep 17 00:00:00 2001 From: Nirbheek Chauhan Date: Sun, 1 Apr 2018 01:28:02 +0530 Subject: [PATCH] sendrecv/js: Don't reuse peer_id across sessions It increases the likelihood of a collision with someone else, and it was an unintended side-effect anyway. --- webrtc/sendrecv/js/webrtc.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/webrtc/sendrecv/js/webrtc.js b/webrtc/sendrecv/js/webrtc.js index 6c86c84..3c928ef 100644 --- a/webrtc/sendrecv/js/webrtc.js +++ b/webrtc/sendrecv/js/webrtc.js @@ -11,7 +11,7 @@ var ws_server; var ws_port; // Set this to use a specific peer id instead of a random one -var peer_id; +var default_peer_id; // Override with your own STUN servers if you want var rtc_configuration = {iceServers: [{urls: "stun:stun.services.mozilla.com"}, {urls: "stun:stun.l.google.com:19302"}]}; @@ -180,8 +180,7 @@ function websocketServerConnect() { if (textarea.value == '') textarea.value = JSON.stringify(default_constraints); // Fetch the peer id to use - peer_id = peer_id || getOurId(); - ws_server = ws_server; + peer_id = default_peer_id || getOurId(); ws_port = ws_port || '8443'; if (window.location.protocol.startsWith ("file")) { ws_server = ws_server || "127.0.0.1"; -- 2.7.4