bjornregnell / sigrid

Help queue web app for lab sessions and tutorials.
Apache License 2.0
10 stars 5 forks source link

improve Beppe functionality, avoid rooms lost in space #22

Closed bjornregnell closed 2 years ago

bjornregnell commented 3 years ago

There is a risk that a beppe login gets lost in space. This issue is opened to explain all this and discuss options. Contribs weelcome!

bjornregnell commented 3 years ago

Dump from Discord (sorry for my southern swedish accent :)):

frodi-karlsson commented 3 years ago

Could a simple solution be to randomly generate and display a pass code on login, which could then be used to "usurp" a room in case of accidental logout? This would avoid adding cookie/authentication functionality as well as malicious usurping of rooms.

DagHemberg commented 3 years ago

Copied and translated from discord: "One option could be to handle rooms in Beppe the same way as in Sigrid, so there could be multiple TAs per room"

This would solve the problem where manually refreshing the page kicks you out of the room as a TA, but also the occasional problem where a TA has forgotten to log out of the room when it was previously used.

bjornregnell commented 3 years ago

It seems possible already to hack the beppe url if you know what to write... (long and tedious to write though) One option would be to give a partly random "secret shortcut" like kaka42 for the url and allow the route beppe/kaka42 to take over, which delegates to the hackable URL. So sigrid just need to create and keep track of a Map[String, String] from random "secret shortcut" to the expanded long URL. Should be a minimal hack not interfering with existing code, and a simple addition to the beppe html + some logic.

bjornregnell commented 2 years ago

this is partly fixed by https://github.com/bjornregnell/sigrid/commit/99767823502c4619fe2844e892c223adf06fce5e so closing this for now. Do you agree @DagHemberg @frodi-karlsson that multiple supervisors in rooms is a good-enough-solution that allows to kick a room by anybody daring to use beppe?