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

Security improvements #78

Open
szokodiakos opened this issue Jan 9, 2019 · 1 comment
Open

Security improvements #78

szokodiakos opened this issue Jan 9, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@szokodiakos
Copy link
Collaborator

  • Require ENV placed ID to open the display view.
  • Include a changing ID in the controller URL, so only people who are there can access the controller
  • Throw out first person in queue after 30 secs of not starting the game.
@szokodiakos szokodiakos added the enhancement New feature or request label Jan 9, 2019
@cxcorp
Copy link
Collaborator

cxcorp commented May 27, 2019

Throw out first person in queue after 30 secs of not starting the game.

Implementing this on the backend would require some pervasive or possibly fragile changes to the state class because of all possible cases.

Do we need to implement this on the backend? On the frontend, this would be quite simple to implement with a timer and page refresh, but is there a threat of someone showing up with a laptop, patching it out and blocking the queue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants