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

step-06 is not working #67

Closed
Im10Anish opened this issue Feb 13, 2018 · 8 comments
Closed

step-06 is not working #67

Im10Anish opened this issue Feb 13, 2018 · 8 comments

Comments

@Im10Anish
Copy link

The app works fine till step-05 but after include step-06 into my code then it shows getUserMedia() error: ReferenceError. When it not working from work directory then I tried directly from the step-06 directory but still same error shows. Can you please look into that.

@nitobuendia
Copy link
Contributor

@Im10Anish Sorry to hear the issues and thanks a lot for reporting.

Could you paste here the whole content of your console logs? That'd help to troubleshoot or identify the issue.

Thank you so much for reporting!

@bedus-creation
Copy link

Yes Step-6 is not working, It says =>
getUserMedia() error: ReferenceError

@nitobuendia
Copy link
Contributor

I was able to replicate:

  1. npm install
  2. node index.js
  3. Access localhost:8080

console.log returned:

Message from server: Received request to create or join room ce3db7fc37d22
main.js:77 Message from server: Room ce3db7fc37d22 now has 0 client(s)
main.js:77 Message from server: Client ID cu9TUnsUDvi5mZEZAAAA created room ce3db7fc37d22
main.js:53 Created room ce3db7fc37d22 - my client ID is cu9TUnsUDvi5mZEZAAAA
main.js:118 Getting user media (video) ...
main.js:48 Server IP address is: XXX

Then opened an alert:
getUserMedia() error: ReferenceError

@nitobuendia
Copy link
Contributor

Error is triggered by: grabWebCamVideo
Full error: Stream error ReferenceError: streamURL is not defined

Working on it.

@nitobuendia
Copy link
Contributor

Proposed #68 to fix it. //cc @samdutton

Thank you so much @Im10Anish and @bedus-creation for your report :)
When #68 is submitted, it should work when you test it :)

@Im10Anish
Copy link
Author

Im10Anish commented Feb 27, 2018 via email

@nitobuendia
Copy link
Contributor

@Im10Anish No worries :) Thanks again for reporting and your help. I've asked @samdutton to review and merge already.

@samdutton
Copy link
Contributor

Thank you @Im10Anish and @bedus-creation!

I've merged #68.

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

4 participants