-
Notifications
You must be signed in to change notification settings - Fork 502
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
Code to notify user when room is full is in available but it does not work! #30
Comments
Could you elaborate on this? What exactly is not working here? |
The problem here is that once two people are connected in a room the server
disconnects, essentially making the room name available again for more to
join. To fix this you would need to keep a persistent connection to the
server even after calls have connected.
…On Wed, Apr 15, 2020 at 5:44 AM Chaphasilor ***@***.***> wrote:
Could you elaborate on this? What exactly is not working here?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#30 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIKTWLIKCHBQEIEP7PXVFMTRMV6XVANCNFSM4MIKQOEA>
.
|
How about instead of a persistant connection, we make the clients, possibly only the 'host' (or innitiator of the call) ping the server every few minutes? Or we could make both clients ping the server and the server hashes each client's IP and links it to the room, so than once one client disconnects from a room, another client can join. We'd need to increase the ping interval, but that shouldn't be an issue. This should guarantee anonymity for the clients and prevent this issue from happening? |
Yes that would work, and be a simple solution. Another option is to try and
get group calling working.
… How about instead of a persistant connection, we make the clients,
possibly only the 'host' (or innitiator of the call) ping the server every
few minutes?
The server just keeps a list of active rooms and removes the rooms after
not getting a ping for a few minutes?
Or we could make both clients ping the server and the server hashes each
client's IP and links it to the room, so than once one client disconnects
from a room, another client can join. We'd need to increase the ping
interval, but that shouldn't be an issue.
This should guarantee anonymity for the clients and prevent this issue
from happening?
|
Wouldn't we still need limit the size of the call? Depending on the topology we use, we can't support a call with 8 people or more... |
I noticed that the following line quotes here should limit users once the room has 2 participants but it does not work.
decentralized-video-chat/server.js
Line 91 in 6101706
The text was updated successfully, but these errors were encountered: