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 script-libp2p.js #2

Closed
wants to merge 9 commits into from
Closed

Conversation

CocoonCrash
Copy link

Updated code to simply set srcObject to the MediaStream directly.
https://developer.mozilla.org/en-US/docs/Web/API/URL/createObjectURL

Updated code to simply set srcObject to the MediaStream directly.
https://developer.mozilla.org/en-US/docs/Web/API/URL/createObjectURL
@victorb
Copy link
Owner

victorb commented Feb 22, 2019

Thanks! I think the last time I tried this, it wouldn't work in Chrome or Firefox. Can you confirm this works with both browsers?

src --> srcObject
Signaling servers changed
@CocoonCrash
Copy link
Author

CocoonCrash commented Feb 27, 2019

Added 2 modifications:

  • "src" changed to "srcObject": my bad
  • Had to change the signaling server url as defaults changed and the former raised a multiaddress format error

Tests I run:

  • Latest versions of Chrome/Edge: can run the webpage with the webcam image showing up, but peers communication doesn't work even if not reporting errors
  • Latest version of Firefox: start to display the webpage but get killed...

Had no time to start debuging all the communication part based on libp2p

Automated http-server launch on "npm run dev"
add "build" directory as watchify doesn't handle directory creation
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

Successfully merging this pull request may close these issues.

2 participants