webrtc-sdk icon indicating copy to clipboard operation
webrtc-sdk copied to clipboard

Double Friends Stream

Open pdxbenjamin opened this issue 5 years ago • 6 comments

With no changes or fancy business. A Downloaded copy of the repo .zip displays two 'friend' video views? I tried a Multicall.html and dial.html in the tutorials folder and they both have double / multiple friend video views when connected? I'm not saying, it is not working, it's double working?! Is that how it is suppose to be? @stephenlb

I've tested with two browsers on my one pc, pc to mobile, pc to pc. All Google Chrome.

pdxbenjamin avatar Jun 21 '19 02:06 pdxbenjamin

Hi! Yes this is working. These HTML files are tests for a self-to-self call. This shows the SDK is working successfully.

stephenlb avatar Jun 24 '19 21:06 stephenlb

If you want to test two devices, use this URL: https://stephenlb.github.io/webrtc-sdk/

stephenlb avatar Jun 24 '19 21:06 stephenlb

@stephenlb I've successfully tested two computers, many browsers and even mobile. Thank you for the link. I don't understand how to cancel the double video feed. Where in webrtc-v2.js can I look for better understand the v2.js inner workings.

pdxbenjamin avatar Jun 25 '19 01:06 pdxbenjamin

May you screenshot the double video feed? I think it may just be the application doing this rather than the SDK.

On Mon, Jun 24, 2019 at 6:40 PM pdxbenjamin [email protected] wrote:

@stephenlb https://github.com/stephenlb I've successfully tested two computers, many browsers and even mobile. Thank you for the link. I don't understand how to cancel the double video feed. Where in webrtc-v2.js can I look for better understand the v2.js inner workings.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/stephenlb/webrtc-sdk/issues/69?email_source=notifications&email_token=AAALBHTNQX55J4C6D2EVD4TP4FZPRA5CNFSM4H2HTFQ2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYOWJAA#issuecomment-505242752, or mute the thread https://github.com/notifications/unsubscribe-auth/AAALBHQSLO7UC4GCHSI4RQDP4FZPRANCNFSM4H2HTFQQ .

-- Stephen Blum - CTO https://www.linkedin.com/in/stephenlb/

stephenlb avatar Jun 25 '19 20:06 stephenlb

This is a fresh download of the repo. Using the multicall.html as a starting point, I changed the pubnub keys to my own. I have not made any other changes to webrtc-v2.js or the .html file. I've shown in my screen shot in chrome dev tools the 'friends' stream shows twice. This is locally of course on my pc, I use firebase if that matters but when it's live testing with another computer, or mobile or tablets every instance is doubled like this. If that is how it is suppose to be for demonstration purpose, cool, but what could I do to not double the friends stream?

@stephenlb

Thank you.

https://imgur.com/a/D0CPDzu

pdxbenjamin avatar Jun 25 '19 22:06 pdxbenjamin

I'm not sure why, again i'm not 100% on the inner workings of the webrtc-v2.js.

This simple change of adding more than the default, audio:true and video:true to the mediaconf has removed the double video.

v2.js

    const PHONE = window.PHONE = config => {
    const PHONE         = () => {};
    const pubnub        = socket(config);
    const pubkey        = config.publish_key;
    const subkey        = config.subscribe_key;
    const autocam       = config.autocam !== false;
    const sessionid     = uuid();
    const mediaconf     = config.media;
    // const mediaconf     = config.media || { audio : true, video : true };
    const conversations = {};
    let   myvideo       = document.createElement('video');
    let   snapper       = () => ' ';
    let   mystream      = null;
    let   myconnection  = false;

...

multicall.js

    let   session = null;
    const number  = Math.ceil(Math.random()*10);
    const phone   = PHONE({
      number        : number,
      media         : { video: { width:1280, height:720 } },
      autocam       : false,   
      publish_key   : 'pub-c-ed7ff8bb-737d-4732-8310-cc69dcd59',   
      subscribe_key : 'sub-c-c43d65b2-9f4d-11e8-84a1-bd3b0340'
    });

I was adding in Xirsys and doing a little bit of styling with css to make the mobile look a bit better. I wanted to add a bit of default sizing to the video feeds and added some constraints to the multicall.html media config and bamo, no more double friends feed. Weird eh. @stephenlb

https://imgur.com/a/cDrsLR4

pdxbenjamin avatar Jun 26 '19 03:06 pdxbenjamin