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

Some precisions #13

Open
yarekc opened this issue Sep 4, 2020 · 5 comments
Open

Some precisions #13

yarekc opened this issue Sep 4, 2020 · 5 comments

Comments

@yarekc
Copy link

yarekc commented Sep 4, 2020

Hey, this project looks awesome. I have some questions

  1. I cannot find the nodejs server part in source code.

  2. is there an online demo so I can test it ?

  3. I guess it works fine with chrome and FF. But did you try it with Safari (it has probably a different mediaRecorder API as usual)

Regards

@StefansArya
Copy link
Member

Hi,

  1. I cannot find the nodejs server part in source code.

There's some example to work with Node.js in the /example folder, I haven't make it as a library for Node.js server. Because I think it's better to build a streaming server with language like C, Rust, or Golang.

  1. is there an online demo so I can test it ?

Hmm, sorry it doesn't have yet. But looks like Heroku now support WebSocket, maybe I should try to make an online demo later.
Actually you can just:

  • Clone this repository and go to the example folder
  • Install the dependencies (npm install)
  • Start the server (node server.js)
  1. I guess it works fine with chrome and FF. But did you try it with Safari (it has probably a different mediaRecorder API as usual)

Thanks for remind me, but sadly me and my friends only have Windows instead of MacOS so I haven't test it on Safari. If you found any error when using Safari browser feel free to fill new issue. Maybe I can fix it by some provided info.

@yarekc
Copy link
Author

yarekc commented Sep 5, 2020

Thanks for answers
I tested some demos:

  • group-call : worked fine (you should however change presenterID: to socketID : it is confusing)
  • video-player : works fine
  • p2p-video-stream : looks like server files are missing. How to run that demo ?

For me this is probably the most interesting demo to test. But I have no idea how. Could you add a readme.txt on how to run it ?

There is a readme.txt that just says "Have bug :("

Regards

@StefansArya
Copy link
Member

Hi sorry for being away because I have some other project to be done.

(you should however change presenterID: to socketID : it is confusing)

Well, both presenter and streamer can have a socketID. I think it's easier to understand since it's pointing the presenter. It could be changed on your implementation for the server.

p2p-video-stream : looks like server files are missing

Yep, my last demo for that still not perfect and have problem with the browser's video decoder. Sometime it's working and sometime it's broken. If you want to try it you can just replace ScarletsAudioStreamer with ScarletsVideoStreamer like on the readme. But I can't guarantee it would work 100% 😅.

I'm planning to redesign it and use WebRTC instead of sending video chunk through the server. But I prefer to redesign the other feature too, and it would cause some breaking changes..

@ahwelgemoed
Copy link

Hi Sorry to JUMP on the back of this - But @StefansArya I pulled the Project and cannot seem to get group call to work. Can it work with just http and local host or must it be https with SSL ?

@StefansArya
Copy link
Member

Hi @ahwelgemoed,
Does the the browser give any error message from the DevTools's console?

It should work on http and localhost, because on the group-call example I setup it up to run on http://localhost:8000, and have tested it on Chromium v88.0.

But after I test it on Firefox, looks like it has a timing problem and the audio got delayed when being used as a streamer, and as a presenter I just realized if it has an error. It will need some investigation for the delayed audio.

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

No branches or pull requests

3 participants