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

CHANGE SERVER FROM LOCAL TO LIVE #70

Open
ghost opened this issue Mar 21, 2018 · 2 comments
Open

CHANGE SERVER FROM LOCAL TO LIVE #70

ghost opened this issue Mar 21, 2018 · 2 comments

Comments

@ghost
Copy link

ghost commented Mar 21, 2018

how can I change the code to move server from local to a live server

@nitobuendia
Copy link
Contributor

Hello @sarathkumar654,

In general, the code is not designed to be ported to a production environment (and it does not intend to be). That being said, technically speaking you can port it to a remote (Node.js) server with a couple big changes:

  1. As you move to your live server, you will need to use secure protocol. This implies using SSL and it's being discussed in fixing step-05 and step-06 enabling ssl in node.js #48. We do want to implement this on the codelab to make the transition easier.

  2. You may require a better signaling system which might involve STUN and TURN servers. Sam discusses this on this article. Our code does gather most of the code required (and Sam explains it there too). However, the servers provided on the codelab are just an example and you will require to replace with yours.

  3. In general, a lot of things are not well designed for production. For example, the coordination of rooms. You will really want to improve the user experience and ensure that joining and changing rooms and connecting peers is done efficiently. There is a few broadcast messages sent on our code that probably will need to just be sent among the peers. This is because the code here is just to be meant to be illustrative.

Those 3 are the ones that comes to mind, but there might be others.
I hope this helps to get you started at least.

// cc: @samdutton

@nitobuendia nitobuendia mentioned this issue Apr 22, 2018
@nitobuendia
Copy link
Contributor

@sarathkumar654 I understand this helped you?

/cc @samdutton

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

1 participant