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

generating meeting and participant-level IDs for communication with rhythm server #13

Open
dcalacci opened this issue Jul 26, 2016 · 1 comment
Assignees
Milestone

Comments

@dcalacci
Copy link
Contributor

dcalacci commented Jul 26, 2016

One issue with the current implementation is that we are using hardcoded meeting IDs (or meeting IDs that are room names) and participant names. We should try to do something like the following:

  1. each new room (even if it is a similar room name to a previous room) is assigned a unique identifier, and rhythm-rtc stores this identifier. if you make a new room with the same name as a previous one, it's overwritten.
  2. each user in a new room is assigned a unique identifier as well. If I join meeting A, my identifier will be different than my identifier for meeting B.
  3. data sent to the rhythm server uses these identifiers for participant IDs and meeting IDs.

Long-term, it would be nice to track participants and meetings over time, but we should first implement sending "correct" data from rhythm-rtc to the rhythm-server, and this is the most straightforward way to do it.

This will mostly be code written in Express on the server-side of rhythm-RTC

@dcalacci
Copy link
Contributor Author

maybe using the easyRTC participant IDs?

easiest way to do this would be:

  • keep a list of all the active rooms in the EasyRTC server
  • each time a new room opens, generate a random UUID
  • when a participant joins a room, they can make an API call to get the random UUID of their room

that UUID is what is sent to the rhythm-server and is the "true" identifier of a room/meeting, rather than the text name.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants